-
Notifications
You must be signed in to change notification settings - Fork 903
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
Deleting document breaks onSnapshot observer in emulator #2197
Comments
@yuchenshi this seems to be an emulator-only issue with clear reproduction steps, could you take a look? |
This seems a lot like firebase/firebase-js-sdk#2967 where @pfiadDi reported a similar symptom. This may be a bug in the Firestore Emulator. Another repro and instructions (credits to @pfiadDi): https://github.com/pfiadDi/firestore-snapshot-bug
|
I can now reproduce this against the Firestore emulator and I have a smaller repro that does not use Firebase Auth. Let me dig more into the the Firestore emulator to see why rules are evaluated that way. |
Fix coming in the next release. |
Fixed by @yuchenshi in #2259 and will be included in the next release |
I'm afraid this is still an issue when deleting the document from the emulator Firestore dashboard. *Edit: Never mind, it looks like that when read access is given based on |
@zanona Thx for your update, it helped and saved time for me |
@zanona I had this issue with the emulator too. Thanks for the hint. |
[REQUIRED] Environment info
firebase-tools:
8.2.0
Platform:
macOS
[REQUIRED] Test case
When deleting a document that has a query on a user auth property, the following error is throw on the snapshot listener. The goal is to observe all invites that has the users email. It works properly when in production, just not in the emulator.
Using these rules
And this basic funciton to observe the collection.
Also have a stack overflow question started: https://stackoverflow.com/questions/61567689/firestore-onsnapshot-rules-throw-error-when-deleting-document
[REQUIRED] Steps to reproduce
[REQUIRED] Expected behavior
The snapshot should always return the query - not throw an error because an item was deleted.
[REQUIRED] Actual behavior
The error
FirebaseError: Null value error. for 'get' @ L91
is thrown.The text was updated successfully, but these errors were encountered: