Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notes cannot login in Firefox Nightly, halts at Profile Server request #3857

Closed
vladikoff opened this issue Jan 9, 2020 · 7 comments
Closed

Notes cannot login in Firefox Nightly, halts at Profile Server request #3857

vladikoff opened this issue Jan 9, 2020 · 7 comments

Comments

@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Jan 9, 2020

The extension halts at the OPTIONS profile server /profile request
image

┆Issue is synchronized with this Jira Task
┆Issue Number: FXA-904

@vladikoff
Copy link
Contributor Author

@vladikoff vladikoff commented Jan 9, 2020

This works fine in Firefox Release, @rfk I think we had an issue tracking this that was fixed?

@vladikoff
Copy link
Contributor Author

@vladikoff vladikoff commented Jan 9, 2020

Related #2411

@rfk
Copy link
Member

@rfk rfk commented Jan 12, 2020

The Origin: null thing is from Bug 1405971. It looks like there's some related bustage in Bug 1607154 which sounds similar to what we're doing here (cross-origin request with Authorization header).

From the debug view, profile-server is correctly echoing back access-control-allow-origin: null in the response headers. But the webextension's origin is not actually "null", so the browser rejects its attempt to send credentials with the request.

We should be able to fix this by setting {cors: "ignore"} on the profile-server, but that requires us to update to the latest hapi.

@data-sync-user
Copy link
Collaborator

@data-sync-user data-sync-user commented Jan 21, 2020

➤ Wil Clouser commented:

Is there a timeline when you need this?  Updating HAPI might be a can of worms

@chenba
Copy link
Contributor

@chenba chenba commented Jan 21, 2020

#2601 is the latest "update Hapi" issue.

@rfk
Copy link
Member

@rfk rfk commented Jan 21, 2020

IIUC the change in Nightly was backed out and Notes login should be working again. I've mentioned Notes as a good testcase for manual QA in the bug since it seems to exercise all the right bits of Firefox that come together to cause problems like this.

So, assuming things have indeed starting working again, I don't think there's a pressing time constraint here.

@vladikoff
Copy link
Contributor Author

@vladikoff vladikoff commented Mar 26, 2020

We should be able to fix this by setting {cors: "ignore"} on the profile-server, but that requires us to update to the latest hapi.

We upgraded hapi and can flip the switch if needed in the future. I checked latest Nightly and Notes login works for now 👍

@vladikoff vladikoff closed this Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants