-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Unable to Authenticate (2FA with google prompt) with Gmail #1356
Comments
Have the same problem ... can use 2FA by copying the code from the Authenticator app, but not when I try to approve the login on my phone via 2FA prompt. |
@bas-jansen Looks like the only cookies that got through have the domain |
Workaround for me was completely disabling Shields on log in. When my gmail loaded, Shields were enabled again automatically. |
Still experiencing this issue on Version 0.56.15 Chromium: 70.0.3538.110 (Official Build) unknown (64-bit) |
@tomlowenthal For what it's worth, I just encountered this issue and was able to workaround by disabling shields on accounts.google.com. Perhaps we should restore the |
Still have a lot of users reporting this (myself included). https://www.reddit.com/r/brave_browser/comments/aeyydr/brave_browser_release_channel_has_been_updated_to/ee28nwl/?context=3 |
@jonathansampson are you able to still repro this on latest release? if so, what 2fa method are you using? |
@diracdeltas I have the latest release 0.58.21 and still experience this issue. |
I added Google Prompt as a 2FA option in my account and tried to use it by browsing to https://accounts.google.com. I got the following error in the developer console:
In the successful case, this request hangs as "pending" until I tap "Yes" on my phone. In the failing case, it immediately returns a 403. If I enable all cookies on I compared the cookies in network traces of a successful v. failing flow and saw the same ones being set. Furthermore, the failing POST request never includes any HTTP cookies so it's unclear to me how blocking third-party cookies affects this. |
@fmarier perhaps it is a referer issue. 3rd party cookie blocking also blocks 3rd party referer (spoofs it to be the origin of the site that it is going to). so for instance if user is on gmail.com and makes an XHR to googleapis.com, the referer would be spoofed from gmail.com to https://googleapis.com/ |
^ to test if this is the fix, you can try adding the relevant domains to |
Fixed in brave/brave-core#1543. I will prepare PRs for the other branches. |
…ve-browser#1356) Logging into Google Accounts with the Google Prompt 2FA mechanism doesn't work unless we send the correct referrer as part of the long-polling XHR to content.googleapis.com.
…ve-browser#1356) Logging into Google Accounts with the Google Prompt 2FA mechanism doesn't work unless we send the correct referrer as part of the long-polling XHR to content.googleapis.com.
@fmarier any QA verification need for this? Could you please add appropriate test plan in the PR? |
@srirambv If you want to, that'd be good. I did test it on all of the branches as I was preparing PRs but I haven't re-tested since. Steps:
|
Verification passed on
Used test plan from #1356 (comment) Verification PASSED on
Verification PASSED on
|
@atmin It should be fixed in the next release. |
@atmin This bug is about 2FA on Google. It looks like you are running into a different problem (it's on GitHub). Would you mind filing a separate issue for this so that we can track it properly? |
Sorry, disregard, false alarm. I started investigating the issue to file a bug. The reason turned out to be I accidently turned off date/time sync on my phone, so the clock drifted enough to cause invalid auth codes. |
Test Case
#1356 (comment) as per @fmarier.
Original Issue
Description
With shields up, users are not able to authenticate (2FA) on GMail
Steps to Reproduce
Actual result:
Token is not accepted. Instead, the following message is displayed:
Expected result:
User should be able to log into GMail
Reproduces how often:
Always
Brave version (chrome://version info)
0.55.10
Reproducible on current release:
Tested on Brave (Core) Beta (0.55.10)
Website problems only:
Additional Information
Reported on Twitter:
The text was updated successfully, but these errors were encountered: