-
Notifications
You must be signed in to change notification settings - Fork 948
[Auth] Fix an issue with the UI interop #4688
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
Conversation
|
Size Analysis ReportAffected Products
|
As of the latest persistence compatibility changes, we do store API keys on |
Done per your suggestion. I've also fixed the |
The Closure SDK makes sure the user belongs to the project by checking the API key on the user and the auth object. We don't store the API key on the user in the new SDK so we were checking this using the app name. But it's a perfectly valid use case to pass users between apps on the same project, so this check was wrong.
This PR also removes an unused rollup plugin that was breaking the compat demo build.