[webapp-container] After cookies expire, user needs to login every single time
Bug #1390350 reported by
Alberto Mardegan
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Unassigned | ||
| webbrowser-app |
Fix Released
|
Undecided
|
Alberto Mardegan | ||
| webbrowser-app (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| webbrowser-app (Ubuntu RTM) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
This happened with the facebook webapp, but it's an issue which could happen with any webapp which integrates with Online Accounts: at some point (after weeks of using the webapp), everytime the user opens the webapp he's asked to login again.
This happens because the cookies in Online Accounts have expired, and even after the user has successfully logged into the webapp, the webapp cookies are being overwritten by the cookies copied over from OA (which are still the expired ones).
Related branches
lp:~mardy/webbrowser-app/per-account-dir2
- Olivier Tilloy: Approve
- PS Jenkins bot: Needs Fixing (continuous-integration)
-
Diff: 603 lines (+159/-157)11 files modifiedsrc/app/webcontainer/AccountsPage.qml (+5/-29)
src/app/webcontainer/WebApp.qml (+1/-0)
src/app/webcontainer/WebViewImplOxide.qml (+5/-0)
src/app/webcontainer/WebViewImplWebkit.qml (+1/-0)
src/app/webcontainer/WebappContainerWebview.qml (+4/-1)
src/app/webcontainer/chrome-cookie-store.cpp (+0/-13)
src/app/webcontainer/chrome-cookie-store.h (+0/-7)
src/app/webcontainer/cookie-store.cpp (+12/-20)
src/app/webcontainer/online-accounts-cookie-store.cpp (+4/-5)
src/app/webcontainer/oxide-cookie-helper.cpp (+5/-4)
src/app/webcontainer/webapp-container.qml (+122/-78)
| Changed in webbrowser-app: | |
| status: | Confirmed → Fix Released |
| Changed in webbrowser-app (Ubuntu): | |
| status: | New → Fix Released |
| Changed in webbrowser-app (Ubuntu RTM): | |
| status: | New → Confirmed |
| Changed in canonical-devices-system-image: | |
| status: | New → In Progress |
| importance: | Undecided → High |
Revision history for this message
| Pat McGowan (pat-mcgowan) wrote | #1 |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Confirmed |
| Changed in canonical-devices-system-image: | |
| milestone: | none → ww05-2015 |
| status: | Confirmed → Fix Released |
| Changed in webbrowser-app (Ubuntu RTM): | |
| status: | Confirmed → Fix Released |
To post a comment you must log in.
Ok to land, important fix