WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
246776
REGRESSION (Safari 16?): Firebase authentication fails: Right side of assignment cannot be destructured
https://bugs.webkit.org/show_bug.cgi?id=246776
Summary
REGRESSION (Safari 16?): Firebase authentication fails: Right side of assignm...
Mehdi
Reported
2022-10-19 15:59:21 PDT
The overall issue is that Firebase communicates with an iframe to coordinate login and it seems the events being passed back from the iframe in Safari 16.1 are different and subsequently cause a problem. I searched around on the tracker and couldn't find anything related. Here is the issue on the Firebase SDK issue tracker:
https://github.com/firebase/firebase-js-sdk/issues/6636
Attachments
Add attachment
proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1
2022-10-20 08:50:40 PDT
Could you please provide steps to reproduce here, ideally a minimized test case without libraries? I don't think that this is actionable as is.
Mehdi
Comment 2
2022-10-21 18:12:16 PDT
I don't have a minimal test case yet as I am an end user of Firebase and merely reporting an error that their library runs into. You can repro it by going to
https://fir-ui-demo-84a6c.firebaseapp.com/
? and trying to login via Google. In the console, you'll see an error at a line like: `const { urlResponse, sessionId, postBody, tenantId, error, type } = event;` where event is a MessageEvent. This runs into the error: TypeError: Right side of assignment cannot be destructured I have tried to repro using a minimal MessageEvent but not been able to. I also tried to run a webkit bisect but the archived builds for Mac don't seem to go back far enough to find a passing case.
Alexey Proskuryakov
Comment 3
2022-10-23 12:18:49 PDT
Thank you for the repro case!
Radar WebKit Bug Importer
Comment 4
2022-10-23 12:19:03 PDT
<
rdar://problem/101479243
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug