[Merge] lp:~mardy/webbrowser-app/cookie-sharing-refactored into lp:webbrowser-app

Alberto Mardegan alberto.mardegan at canonical.com
Tue Aug 18 12:25:57 UTC 2015


The proposal to merge lp:~mardy/webbrowser-app/cookie-sharing-refactored into lp:webbrowser-app has been updated.

Commit Message changed to:

Provide a new cookie sharing service for collaborating webapps. This helps share authentication tokens between apps using the same domain / identity provider.

Provide a new cookie sharing service for collaborating webapps. This helps share authentication tokens between apps using the same domain / identity provider.

Basically, every time that a webapp gets unfocused, it sends all of its cookies to the services, which stores them in a json file (whose path depends on the account provider and account number, but is anyway found under ~/.cache/webapp-cookie-service/). Conversely, when a webapp gains focus, it asks the D-Bus service to get the cookies, and stores them into the webview.
The cookies are always transferred via D-Bus calls: the webapps don't have direct access to the cookie file.

We are aware that this solution is not immune from race conditions; testing will tell whether they really have an bad effect on the functionality.

For more details, see:
https://code.launchpad.net/~mardy/webbrowser-app/cookie-sharing-refactored/+merge/262876
-- 
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~mardy/webbrowser-app/cookie-sharing-refactored into lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list