-
-
Notifications
You must be signed in to change notification settings - Fork 344
Signin With Redirect not working #1621
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
Comments
It's happening to me |
Same for me |
Same issue. <script setup lang="ts">
import { getRedirectResult, signInWithRedirect, GoogleAuthProvider } from 'firebase/auth'
import { useFirebaseAuth } from 'vuefire'
const auth = useFirebaseAuth()
function signInWithGoogle() {
signInWithRedirect(auth, new GoogleAuthProvider())
}
</script> |
@Johnnyevans32 , this fixes the issue for me: https://stackoverflow.com/questions/77270210/firebase-onauthstatechanged-user-returns-null-when-on-localhost |
This is more of a firebase issue, not a vuefire issue. I've been experiencing this for years in development. What I do is something like this in my imports, then toggle the comment when I need to login to test. import {
GoogleAuthProvider,
// signInWithPopup, // use this with localhost, still might need to refresh or manually update address bar
signInWithRedirect,
} from "firebase/auth"; |
Reproduction
https://github.com/posva/nuxt--vuefire-example-spark-plan
Steps to reproduce the bug
trigger signInWithRedirect(provider) from firebase auth.
redirect to the oauth provider (e.g., twitter).
sign in successfully.
user is redirected back but not logged in.
no persistent auth state or missing result from getRedirectResult().
Screenrecorder-2025-04-16-03-29-39-789.mp4
Expected behavior
after returning from the oauth provider, calling getRedirectResult() should yield the current user state and the auth state should persist across reloads.
Actual behavior
the app reloads and useCurrentUser() returns null
getRedirectResult() doesn't resolve the user
persistence appears broken, even when set to local
Additional information
signInWithPopup() works fine
The text was updated successfully, but these errors were encountered: