-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
ASWebAuthenticationSession error when using GIDSignIn #3791
Comments
Can you share the stack trace of the crash? The error log at the bottom is a non-fatal error. |
@morganchen12 it does not crash for us, it just fails with the error posted above. |
Yes my mistake the error is being called in More on the error: line 90 of OIDExternalUserAgentIOS.m
However, this code isn't ideal. OIDExternalUserAgentIOS should ask for a window rather than trying to derive it from the ViewController that is provided. |
@renkelvin based on the error posted above it looks like we're misusing AuthenticationServices. Can you investigate? |
@morganchen12 Firebase Auth doesn't depend on AuthenticationServices. It seems an issue with GSI. |
Looks like this was reported earlier at b/139778942 (Google-internal only). |
@morganchen12 b/139778942 should only impact binary installs. If the same thing is happening from a CocoaPods install, it's another issue. @ajkolean Please confirm that you're using the most recent version of GoogleSignIn - 5.x |
Yes I am, this is from my Podfile.lock:
|
There is a compatibility issue with AppAuth 1.1.0 openid/AppAuth-iOS@d16caab |
how to fixed ???? :( |
You'll get AppAuth 1.2.0 now if using CocoaPods, which should fixed this issue. |
Why is this issue closed? There is still no new binary release which fixes the issue. |
@tkrajacic Good point. Thanks! This issue was missing a milestone. I've updated the issue to include the 6.10.0 milestone which is stabilizing now and should publish next week for CocoaPods, Carthage, and the Firebase binary release. I can't speak for the GoogleSignIn releases. Our policy in this repo is to close issues when they're fixed in source and do release tracking based on the associated milestone. |
Even after release of 6_10_0 we still do observe the same behaiour on devices running iOS 13. After invoking |
@vetrovosk Please confirm that you updated the AppAuth.framework to the version 1.2.0 from the zip download, did a clean rebuild, and removed DerivedData( |
Yes, I confirm that I've just made a fresh clone of google signin sample app, opened it in Xcode 11, changed client id in app delegate, changed I've also removed I also observe that files inside zip distribution are dated with Here is the sample application output:
|
Thanks @vetrovosk. It looks like the AppAuth fix doesn't get applied in our Xcode 10.1 built binary release distributions. We'll investigate solutions. In the meantime, the workaround would be to integrate the AppAuth source or rebuild a binary framework with Xcode 11. |
Yes, I even think that I've mentioned that their fix uses compile time checks that don't get applied when built against older versions of ios sdk in the support Case 00011522 submitted to [email protected] :-) Thanks! |
According to the GoogleSignIn team, the 5.0.1 zip file downloadable from https://developers.google.com/identity/sign-in/ios/sdk has the fix. I'll leave this issue open until we have a resolution for the Firebase zip and Carthage distribution. |
Is anyone still running into this issue with latest Firebase 6.10.0? Seems still occurs - perhaps that's what you're referring to @paulb777 that everyone still waiting on full resolution in Firebase ditro? Perhaps in a 6.11.0 upcoming release? |
@NathanWalker We don't have a mechanism in Firebase to support different binaries for both Xcode 10 and Xcode 11, so we're planning to drop distribution of the GoogleSignIn binary and instead reference the GoogleSignIn team's official distribution channel at https://developers.google.com/identity/sign-in/ios/sdk |
@paulb777 when we try to use GoogleSignIn SDK instead of
|
Thanks for the report @vetrovosk A likely workaround would be to stop linking the GTMSessionFetcher.framework from Firebase. |
I'm still having this issue, any timeframe for when this might be fixed? Thanks. |
@peteyblah To summarize, the issue is fixed for CocoaPods users. For binary distro users, you need to use one of the workarounds discussed above. We're targeting a binary distribution solution for the M59 release, which is roughly three weeks from now. |
I've read through the thread, I've tried using the package from the GoogleSignIn team with the 5.0.1 zip, but end up having the same error as mentioned above (even after a clean build and clearing derived data), that I get with the complete Firebase package. |
This should be fixed in today's Firebase 6.12.0 in which the zip and Carthage distros are built with Xcode 11 instead of Xcode 10. cc: @maksymmalyhin |
I'm afraid that I'm experiencing the same issue even with 6.12.0 release. Calling |
@olejnjak Thanks for the report and sorry about that. It turns out we had a problem with the release scripting and that the Carthage distribution was still built with Xcode 10. We plan to fix in the next release. In the meantime, a workaround for Carthage users is to use |
Firebase 6.13.0 was published to Carthage moments ago and was packaged with Xcode 11. This should be resolved now, please re-open if you still run into issues. |
[READ] Step 1: Are you in the right place?
YES
[REQUIRED] Step 2: Describe your environment
My Podfile.lock is
[REQUIRED] Step 3: Describe the problem
Trying to sign in with Google causes crash.
The text was updated successfully, but these errors were encountered: