-
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
Crash in FIRInstanceIDCheckinService.m on iOS 13 when app is a new install #3842
Comments
This comment has been minimized.
This comment has been minimized.
This only happens on the new iPhone 11 sims. I'm able to run my app just fine on XR 13.1
|
@kevinchau Thanks for the report. Please confirm what's installed. Symbols from |
@paulb777 Thanks, here are the Firebase pods that I have:
|
This actually crashes on every device with iOS 13 - if I delete the app, and re-install it, it will crash on first run and subsequent runs. But if updating from iOS 12 and an older version of the app, it will run |
@kevinchau can you run |
@morganchen12 no problem, here you are:
Not sure if this helps, but including it anyway: |
@chliangGoogle looks like a keychain-related crash in InstanceID. Does this look like a regression? |
Just a note, I'm able build and run using the iOS 12 SDK on Xcode 10.3. It works fine there. |
@kevinchau Is it only happening on real devices? I can't reproduce on simulators using Xcode 11 GM |
@chliangGoogle both. It's only happening on new installs though. |
@morganchen12 FCM with InstanceID works without crash, unlikely a regression. Seems like a complication with the new remote config and perf, I will do more testing on that. |
@kevinchau I'm not able to reproduce yet, can you share with me your sample code? especially the way you call remote config during app start? |
I'm not sure this is a InstanceID issue, from the stack trace, looks like issues are happening inside remote config thread, although @kevinchau manually highlighted the IID thread. I only see errors occur inside RC call here:
|
@chliangGoogle I can send you some code samples, I'd prefer not posting it to the public. |
SG, feel free to DM me. |
@kevinchau is it possible to repro without the Perf SDK? |
@chliangGoogle looks like Github doesn't have DM feature anymore, have an email? |
I can't repro it anymore - I was having a build issue with Xcode 11 and had to Pod deintegrate and reinstall. It appears to have went away. Thanks for the help, and sorry it was not an bug on your end. |
Glad it's gone. Might have been a variation of #3344 if you had old pods before deintegrating. |
[REQUIRED] Step 2: Describe your environment
[REQUIRED] Step 3: Describe the problem
Crashes on Launch
Relevant Code:
The text was updated successfully, but these errors were encountered: