enh(#861): Redirect back to requested page after authentication #870
Conversation
andychukse
changed the title
|
Really looking forward to this feature |
andychukse
force-pushed
the
enh/861-redirect-back-to-requested-page-after-authentication
branch
from
d84b51f
to
f3ccf91
Compare
commit: |
There was a problem hiding this comment.
Love this feature! Thank you so much for your PR ❤️
One suggestion I would have is to respect the addDefaultCallbackUrl option inside the middleware and only add the callback URL to the redirect if it is enabled!
This should be a pretty straight forward adjustments, where you check if globalAppMiddleware.addDefaultCallbackUrl is set before adding query.redirect to the navigateTo call!
There was a problem hiding this comment.
Functionally approved 🥳
@phoenix-ru, please do a code review 😊
There was a problem hiding this comment.
@andychukse Thanks for your PR! I think it will bring local provider in parity with what authjs already does in terms of redirects
|
PR Released in https://github.com/sidebase/nuxt-auth/releases/tag/0.9.3 🥳 |
🔗 Linked issue
#861
❓ Type of change
📚 Description
Add functionality redirect user back to requested page after authentication
Added a redirect query parameter to auth middleware navigateTo when a guest visits authenicated page.
This will help users redirect back to previous page after sign in.
Currently when guest tries to access a protected page they are redirected to
/loginpage and after login they are redirect to defaultcallbackUrlunless they specifycallbackUrlin thesignInfunction.This feature will help automatically redirect a user back to the page they requested after login. For example, if they requested to access
'/profile'and they are redirected to login page, after successful login, they will be redirected back to'/profile'. They can also override this by providing acallbackUrlin the signIn function as it is currently📝 Checklist