-
Notifications
You must be signed in to change notification settings - Fork 878
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
FIRESTORE (10.12.0) INTERNAL ASSERTION FAILED: Unexpected state #8250
Comments
Looking into the Firebase logs, the error seems to occur after Our app uses I wonder if there would be any legs in not having a tab manager if that will result in no calls to This is also similar to what is described by another GitHub user here I don't know a lot about the internals of the tab management so might be completely wrong here. Sample Log
@wu-hui I would be Happy to share the full log with you privately if you could let me know the best way to do that. |
@vojto Perhaps that's a different issue, this has always been limited to just iOS for us. |
I think the multitab log might be unrelated, this seems to stem from a underlying webchannal message. |
Fair enough, not an area I've dug too deep into. @wu-hui what steps can I take to help try and move this forward? |
Still seeing this issue on the latest version. I have logs I can share if that would be useful? |
@luke-rogers I would like to review the logs. You can share the logs through a private GitHub repo (shared with me), through a Cloud support ticket, or through a Firebase support ticket. If creating a ticket with either Google support channel, please reference this GitHub issue so the support staff knows to route the ticket to my team. Thanks! |
I encounter the same problem. Usually this is combined with
The problem is always resolved by loggin out (clearing local storage). EDIT: I created a new issue, as I am not sure if the underlying problem is the same. |
@MarkDuckworth I've shared the logs through a Firebase support ticket and via a private repo Let me know if you need anything more from me. |
@MarkDuckworth did you get access to the logs? Support said they forwarded them on. |
@luke-rogers, I received the logs. Thank you for sharing. On initial review, I don't see an obvious cause, but we will give it some deeper investigation as soon as we can. I've also started work on a PR to help us get error codes from these internal assertion failed messages. Do you also have a reproducible example that you can share with us? |
@MarkDuckworth Thanks for the update. PR looks good. Hopefully, you will be able to merge that. We would be happy to test a pre-release version of this if that would still be helpful? Unfortunately, we do not have a way of reproducing this ourselves at the minute. I have added some links to stack traces from Sentry to the readme of private repo I shared with you. |
@luke-rogers Mark is out of office and asked I take a look at logs. Could you grant access to private repo, to myself @tom-andersen and maybe some other members of our team such as @wu-hui ? Thanks. |
@tom-andersen added yourself and @wu-hui to the repo. Still struggling with a reproduction but if there is anything else I can try and help with please let me know. |
Update: The error with text The workaround might be to enable multi-tab support, since Firestore SDK will then handle this error by entering a non-primary role until IndexedDB resolves the problem, instead of treating this like a fatal error. That this happens at all, is a little concerning, and might have more to do with the AppleWebKit. |
@luke-rogers Are you able to enable multi-tab and see if that resolves the problem? |
@tom-andersen we will enable multi-tab and report back. Hopefully doing a release today so should have some initial feedback by the end of the week. |
Hey @luke-rogers. We need more information to resolve this issue but there hasn't been an update in 5 weekdays. I'm marking the issue as stale and if there are no new updates in the next 5 days I will close it automatically. If you have more information that will help us get to the bottom of this, just add a comment! |
We released yesterday with the change to enable multi-tab. We will have more of an update within the next few days as to whether it's made a difference. |
Unfortunately enabling multi-tab has not resolved this issue. I am still trying to access the impact on the end user experience but we have managed to capture some logs for the error. I've added those to the GitHub repo previously shared. The new log files for this error with Firebase version 10.12.4 and multi-tab enabled are: |
Operating System
iOS 17.4.1
Browser Version
Mobile Safari UI/WKWebView (AppleWebKit/605.1.15)
Firebase SDK Version
10.12.0
Firebase SDK Product:
Firestore
Describe your project's tooling
Angular web app running on iOS via Capacitor
Firestore offline persistence enabled with unlimited cache size
Describe the problem
For a long time now we have been plagued by this iOS-only issue and have received many reports from users that it's causing the app to stop working and forcing them to restart.
We have captured some Firebase debug logs of this error which I am happy to share privately.
This seems similar to the following issues:
Example Errors
Example Stack Trace
Steps and code to reproduce issue
Steps currently unknown but possibly related to the app going between the background and the foreground over time.
The text was updated successfully, but these errors were encountered: