Closed
Bug 1869641
Opened 5 months ago
Closed 4 months ago
Listen to oauth_login webchannel message, and trigger oauth complete
Categories
(Firefox :: Firefox Accounts, task, P3)
Firefox
Firefox Accounts
Tracking
()
RESOLVED
FIXED
123 Branch
Tracking | Status | |
---|---|---|
firefox123 | --- | fixed |
People
(Reporter: teshaq, Assigned: teshaq)
Details
Attachments
(1 file)
With https://bugzilla.mozilla.org/show_bug.cgi?id=1866143 completed, we have all the building blocks needed to hookup the OAuth flow with the current flow of the browser
This bug will add a pref that when enabled, the browser will return an OAuth URL instead of the fx_desktop_v3 one, and once web content sends back the OAuth code and state, the browser will authenticate the user and persist their sync keys.
Assignee | ||
Comment 1•5 months ago
|
||
- Adds new pref disabled by default that enables the oauth flow
- When the pref is enabled, the URLs returned by FxA are oauth urls
- Adds listening to oauth login web channel message from FxA
Pushed by teshaq@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/686fad3c7efc Adds oauth handling for fxa behind pref. r=skhamis
Comment 3•4 months ago
|
||
Backed out for causing bc failures on browser_UITour_sync.js
Flags: needinfo?(teshaq)
Assignee | ||
Comment 4•4 months ago
|
||
My bad! Fixed the breakage and ran try https://treeherder.mozilla.org/jobs?repo=try&revision=6d743bc42e923d8155598b152f002721cf0b704a&selectedTaskRun=QrmHjOfeR8eqJFqXWt8XZA.0
none of the failures there are related, hopefully sticks this time (otherwise It'll have to wait to 124)
Flags: needinfo?(teshaq)
Pushed by teshaq@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/53a0a5eed5d7 Adds oauth handling for fxa behind pref. r=skhamis,markh,sync-reviewers
Comment 6•4 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
status-firefox123:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•