-
Notifications
You must be signed in to change notification settings - Fork 892
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
Firebase return document not exist after clear cookies with IndexedDb persistence enabled. #6591
Comments
Can you clarify what "clear website cookies from Chrome address bar" means? Do you mean url params? Or do you mean clearing cookies with Chrome dev tools? Or do you mean selecting "Clear Browsing Data" from the Chrome 3-dot menu? If it's the last choice it might be wiping all local storage, including IndexedDB. Then it's not able to find the document in the local cache. But not sure why it's not falling back to checking the Firestore backend, I guess I'll leave that for the Firestore team. |
Hi, Can you provide code snippet you used to reproduce this behaviour? |
` const appCheck = initializeAppCheck(app, { // clearIndexedDbPersistence(getFirestore()); onSnapshot(doc(getFirestore(), Please note as I tested, the test document path must be a path ruled to only be able to access by certain account, In my case I set rule to check on custom account metadata. So to reproduce this on your side, you must sign in with correct account first, load the protected document let it saved in local indexDB, and then clear cookies and sign in again, In my case, I signed in with another newly created account but with the same custom metadata (so same rights to access firebase), then I will have that wrong document status issue. If only unmark the above line clearIndexedDbPersistence, and then this issue will gone. |
Hi, With |
Hi, because all my documents is protected by rules to limited to sign in account read only, so, you can not check the issue with out sign in first. I agree there is no such use case, 99.99% of the user will not do clear cookies from the browser, but because once this happened, will leave some kind of bad status in IndexDb and even refresh page not be able to solve it, so the effected user will be blocked from using the site, which I think is significant. |
Thank you for your context and spending time help us reproduce this problem! We have reproduced it and captured it to the backlog (b/249145220). However given the rare case, we won't prioritize fixing this. Thank you again for your understanding. |
I am experiencing the same issue when the user clears their recent browsing history in Chrome. For us, the impacted user is also blocked from using the entire site. For us, it is with enableMultiTabIndexedDbPersistence enabled. |
Hi @fypmvip , Could you please file another ticket for your issue and provide detailed reproduce steps? |
Hi, This feature is been tracked internally, I will close this github ticket and reporter will be notified in the future when the feature gets implemented. |
Hi, The team decided to keep the ticket open until the feature is implemented or bug is fixed for external visibility. |
Hi @cherylEnkidu , @hsubox76 |
Same here, Guys this is a very concerning problem for production apps. We are just not going to be able to use persistence until it is fixed. Please check it again... |
[REQUIRED] Describe your environment
[REQUIRED] Describe the problem
Firebase return document not exist after clear cookies with IndexedDb persistence enabled.
Steps to reproduce:
This is only happening with enableIndexedDbPersistence enabled.
The text was updated successfully, but these errors were encountered: