-
Notifications
You must be signed in to change notification settings - Fork 475
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
ITMS-90809: Deprecated API Usage - FirebaseUI/Google #771
Comments
This comment has been minimized.
This comment has been minimized.
@renkelvin, iirc there aren't any code changes necessary, I just need to bump the dependency version in CocoaPods. Can you confirm? |
If FirebaseUI/Google code base is compatible with GoogleSignIn 5.0.0, then yes it is just a dependency bump in CocoaPods |
I'll take care of this today. Thanks for reporting! |
Have you got a chance to look at it, is the dependency bump worked or it needs more integration work? |
I'm also wondering about the state of this issue since the dependancy is not bumped to 5.0.0 yet. |
Apologies for the delay, this has been released. Please update your FirebaseUI version and let me know if this is still an issue. |
Installing FirebaseUI with CocoaPods, I still have GoogleSignIn version 4.4.0 instead of 5.0.0. Does anyone have the same issue? And If I run the following command in the terminal: I got this answer: Apple will not accept my app with the UIWebView API. |
I updated to FirebaseUI 8.1.0 and GoogleSignIn got updated to 5.0.0 as well. The grep match for UIWebView in GoogleSignIn has disappeared. Make sure you use |
Ok, thank you! Solved! |
Thanks it is solved |
./Pods/GoogleSignIn/CHANGELOG.md:- Supports sign-in via UIWebView rather than app switching to a browser, when grape UIWebView Using GoogleSignIn 5.0.0 |
I'm using: What should I do? |
Run |
GoogleSignIn is still using AppAuth, which includes UIWebView. When would this be resolved? |
Appstore is keep giving warning after uploading App to Appstore - Dear Developer, We identified one or more issues with a recent delivery for your app, "XXXX. Your delivery was successful, but you may wish to correct the following issues in your next delivery: ITMS-90809: Deprecated API Usage - Apple will no longer accept submissions of new apps that use UIWebView as of April 30, 2020 and app updates that use UIWebView as of December 2020. Instead, use WKWebView for improved security and reliability. Learn more (https://developer.apple.com/documentation/uikit/uiwebview). After you’ve corrected the issues, you can upload a new binary to App Store Connect. |
Versions higher than GoogleSignIn 4.4.0 produce errors when building, version 4.4.0 works fine without any error, but the problem is that it uses UIWebView, I would like UIWebView of 4.4.0 replaced with WKWebView. |
Same here, some update? |
AppAuth references UIWebView in a header comment only; you can either appeal your app rejection to Apple or you can remove the header comment manually and resubmit. See this issue: openid/AppAuth-iOS#449 |
5.0.2 is available, just update your Podfile to Here is how to migrate from 4.x: https://developers.google.com/identity/sign-in/ios/quick-migration-guide |
@kazunetakeda25
I'm having the exact same issue, have even tried importing this repo and use 5.0.2 based on googlesamples/google-signin-unity#122 but still no luck. Any help on this? |
Hi,
The latest FirebaseUI/Google (8.0.4) is using GoogleSignIn 4.4.0 (and not 5.0.0) and is not compliant anymore with the new Apple policies because of use of UIWebView API.
When trying to submit the app to Apple on AppConnect, they drop a no go warning message, preventing from being reviewed successfully :
ITMS-90809: Deprecated API Usage - Apple will stop accepting submissions of apps that use UIWebView APIs . See https://developer.apple.com/documentation/uikit/uiwebview for more information.
Steps to reproduce :
but when forcing GoogleSignIn to 5.0.0 with
pod 'GoogleSignIn', '5.0.0'
then
FirebaseUI/Google needs to be compatible with the latest release of GoogleSignIn 5.0.0 already available as a pod.
The text was updated successfully, but these errors were encountered: