about:newinstall doesn't actually sign the user in to sync

RESOLVED FIXED in Firefox 67

Status

()

defect
RESOLVED FIXED
2 months ago
22 days ago

People

(Reporter: rfkelly, Assigned: mossop)

Tracking

unspecified
mozilla67
Points:
---

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox65 unaffected, firefox66 unaffected, firefox67+ fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 months ago

Thanks to :rfeeley for noticing this and helping work through it on slack. Steps to reproduce:

  1. Trigger the about:newinstall page to indicate migration to a separate profile for nightly. (I don't know the "right" way to do this, but browsing directly to about:newinstall?endpoint=https%3A//accounts.firefox.com/ seemed to do the job)
  2. Enter your email in the form to connect to sync, which will open the FxA login page in a new window
  3. Complete the FxA login flow
  4. Click the hamburger menu in the browser and observe whether the user is signed in

Expected result: I'm signed in to sync in the browser.

Actual result: The hamburger menu still shows "sign in to sync", indicating that I was not successfully signed in to the browser.

What I believe is happening, is that the new tab opened in step (2) is missing the context query parameter. It should include context=fx_desktop_v3 so that the page knows how to communicate login state back to the browser.

The patch from Bug 1518632 declares a $CONTEXT variable with the correct value here:

https://hg.mozilla.org/mozilla-central/rev/a5ab8fa035ce#l6.14

But its value doesn't seem to be used when constructing the URL for FxA here:

https://hg.mozilla.org/mozilla-central/rev/a5ab8fa035ce#l6.55

Manually adding it to the URL of the tab opened at step (2) fixes the issue for me.

(Assignee)

Updated

2 months ago
Assignee: nobody → dtownsend

Comment 2

2 months ago
Pushed by dtownsend@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/36a3197ce9c6
Include missing context parameter in the Firefox Accounts sign in page url. r=stomlinson

Comment 3

2 months ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
QA Whiteboard: [qa-67b-p2]
You need to log in before you can comment on or make changes to this bug.