-
Notifications
You must be signed in to change notification settings - Fork 290
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
"Missing or insufficient permissions" when upgrading above 9.16.0 for security rules #710
Comments
That's possible. Are you using the default Firebase app? Could you try removing all properties except |
@kmjennison You might be onto something here, when I remove all other properties I'm getting this error:
|
@kmjennison I created a super minimal reproduction in my fork https://github.com/MvRemmerden/next-firebase-auth. In the example folder, I only added one file ( I suppose that means the problem must have been introduced somewhere in the changes between 9.16.0 and 9.17.1: firebase/firebase-js-sdk@[email protected]@9.17.1 These are the database content and rules:
I can also add my firebase configuration, but then I would have to make the repository private and invite you. Just let me know what you prefer. |
Interestingly I just noticed there is one way to make the content under However, as soon as you refresh the page, you will from then on always get the same |
Sadly it does not look like this issue got fixed by #716 😞 |
@kmjennison I've been starting to look into it, as that switch from As you have a way better understanding about the flow and connections, do you have any pointers or assumptions what the underlying problem might be, that could help a lot speed up the investigation process. |
@MvRemmerden I don't, unfortunately, and don't have much spare time to dedicate at the moment. Sorry about that! I'll try to answer questions and jump in when I can. |
@kmjennison I just noticed #669, where I think you started to explore this already. Is this in Draft mode because it just needs to be finished off but already works, or did the changes there not end up fixing the problems when upgrading to 9.17.1? |
I wasn't able to identify the issue (likely something with bundles/builds), which is why we switched to recommend explicitly initializing Firebase: #614 (comment) |
After upgrading
next-firebase-auth
to 1.0.2 and following the new way of initializing the firebase app before NFA, I now get "Missing or insufficient permissions" if I use firebase 9.17.1, but it works as expected when I use 9.16.0.Did this maybe not get caught in the example app, because there are no security rules?
The text was updated successfully, but these errors were encountered: