The server sends a TTL value when initializing a session. Currently we re-init the session ONLY on e.me init - meaning when the homescreen loads. Need to re-init the session whenever it expires. Also make sure it doesn't interfere with user action - so if a user requests something and their session has expired, wait until they get a response and only then call the session init request.

Branch: *session-reinit* https://github.com/EverythingMe/gaia/compare/enhanced...session-reinit Commit: 7bd3870aa5c8de4a395fabf4173535dcb9747431 https://github.com/EverythingMe/gaia/commit/7bd3870aa5c8de4a395fabf4173535dcb9747431

Testing the branch I see that a "no connection" message appears often. Click "TV" you get the message for a second and then results appear. Load more - Message appears at the bottom and then disappears, showing results.

Fixed and pushed.

Just making a note - this patch also handles reporting cached icons on session init. Added comments on Github.

merged to master 9b118f3b66e4bfe77564af9cdcff886de7b8c004 https://github.com/mozilla-b2g/gaia/commit/9b118f3b66e4bfe77564af9cdcff886de7b8c004

I was not able to uplift this bug to v1.2. If this bug has dependencies which are not marked in this bug, please comment on this bug. If this bug depends on patches that aren't approved for v1.2, we need to re-evaluate the approval. Otherwise, if this is just a merge conflict, you might be able to resolve it with: git checkout v1.2 git cherry-pick -x -m1 9b118f3b66e4bfe77564af9cdcff886de7b8c004 <RESOLVE MERGE CONFLICTS> git commit

John, looking in to the merge conflict, we would rather the patch not land in v1.2. Current E.me code has undergone major refactoring and so landing this patch would mean redoing the needed changes and introducing unnecessary risk. The patch is not critical as it influences mostly E.me servers.

Actually, amongst the bugs the this patch fixes is one that is crucial. It is described and has a v1.2 specific patch in bug 932248 that should get nominated.