-
Notifications
You must be signed in to change notification settings - Fork 892
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
auth().signInWithPopup failing for Facebook on Firefox #1003
Comments
Same here U_U |
I have the same problem ... :c |
Same problem also for me. |
Same for me. |
Same problem here!... On the bright side we can always disable the add-on |
I'm also seeing this for Github in Firefox for Firebase SDK version: 6.0.2 const authenticate = provider => {
console.log('Logging In');
const authProvider = new firebase.auth[`${provider}AuthProvider`]();
firebase
.auth()
.signInWithPopup(authProvider)
.catch(err => {
console.log(err);
});
}; {
code: "auth/popup-closed-by-user"
message: "The popup has been closed by the user before finalizing the operation."
} Any insight would be helpful. |
Still happening |
And Still happening |
@rommelpe the original issue is specifically referring to Firefox version 61 with Facebook container. I've checked your test case and the problem persists in Firefox 70 / 71. |
Exactly. As @theprojectsomething suggested. |
@theprojectsomething @rootux ops missed that, my bad. After adding Facebook container, I ran into the same issue. It seems that it is working as expected as the plugin itself says:
It would be helpful if you could reach out to them. They may be able to provide some points to resolve the issue. |
@rommelpe thanks, yes the plugin is definitely working as expected - it's fbauth that's not. Currently fbauth simply fails silently (albeit with a popup error). Given the proliferation of blockers and contextual containers I guess a basic suggestion would be to return a clearer error that doesn't require a lot of research. Building up from that, a message in the oauth window explaining that it is unable to communicate with it's parent, along with some instructions to the user on how to resolve things (perhaps providing a link to be pasted into the original context?) would be a great outcome. |
@rommelpe Thank you. I believe this bug is serious then what originally reported. As @theprojectsomething suggested, a message in the |
Thanks for all the inputs. While this is currently an expected limitation, we'll treat this as a feature request and keep this open. |
Any updates on this or possible workarounds please? Our firebase app is not usable when shared over facebook (mobile user opens it in facebook container). |
Any update? My user is hitting the same issue. |
This seems like an issue with FF or the Extension itself, and i don't think either will try to fix this. |
It's beyond the extension and happens to many users that does not have that
extension installed
…On Sat, 11 Apr 2020, 10:29 Tzah Mazuz, ***@***.***> wrote:
This seems like an issue with FF or the Extension itself, and i don't
think either will try to fix this.
Tell your user(s) to disable the Facebook container extension.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1003 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAH4CTVR5TQXEZD6QLUIKI3RMAL6TANCNFSM4FJT2GPA>
.
|
Same thing happens with Google now |
@olso can't reproduce the issue with Google sign-in (or in Chrome if that's what you meant) .. unless there is a google container installed? Assume that would break Firebase in more ways than just auth. |
Hi All, this behavior is still reproducible using the latest version (8.6.1) of the SDK. If your Mozilla Firefox browser has Facebook Container addon or extension enabled, Firebase Facebook Authentication will fail. I've filed a bug for internal tracking. We can't provide timelines or details, so keep an eye out on our release notes or this issue thread to get updates in the future. |
For reference the internal bug is b/188185399. |
Hey! Any help will be good. |
@eidan66 that looks like the popups have been blocked, it's not caused by the Facebook container. If you click the page settings icon, directly to the left of the url, you'll be presented with a permissions overlay where you can allow popups. Alternatively, follow these official instructions to resolve (note that the UI presented in the docs is old) |
I have fixed this issue. The problem is that I was triggering the popup on click of a button but I was not adding the type="button" attribute in my tag. So what I did is following, I added type="button" and voilaaaaa, It fixed my problem. Sign In With Google |
@verma1019 this doesn't work for me, we had this since the beginning |
Have you been able to solved it? |
@verma1019 if In my case I use |
I solved this issue for myself today (for firefox mobile). The problem seems to stem from the way firefox detects malicious popups: If the popup was not opened directly as a result of some user action, then the popup is temporarily blocked until the user allows it through. In my case, I was generating a recaptcha code before opening the popup, and this was apparently what was causing the popup to be initially blocked. After I changed the code to open the popup first, then do the recaptcha stuff, I was no longer getting any dialogs to allow popups/signinwithpopup was now successfully running. EDIT:As per this comment, it seems the default action by firefox is to block all popups (...which do not result from a user action. However, it seems their heuristic for detecting this may need some tweaking). However, due to this other bug firefox does not remember the choice of the user, so it ends up never allowing popups to display. |
Describe the problem
Firefox's Facebook container seems to be causing the
auth().signInWithPopup
flow to fail at the redirect and close stage.The popup window logs the warning "Scripts may not close windows that were not opened by script."
The SDK returns "The popup has been closed by the user before finalizing the operation."
Realise this is due to an addon, but it seems like it might be a fairly common issue amongst Firefox users given its popularity. It's definitely causing me heartache. I'd file a bug with Mozilla but I'm not sure of the exact details, i.e. is this likely to effect most oauth implementations?
Steps to reproduce:
signInWithPopup
auth flow, using Facebook as the auth providerRelevant Code:
The text was updated successfully, but these errors were encountered: