Skip to content
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

[1.x] Redirect to intended path after login #39

Merged
merged 1 commit into from
Feb 11, 2021
Merged

[1.x] Redirect to intended path after login #39

merged 1 commit into from
Feb 11, 2021

Conversation

martijnb92
Copy link
Contributor

Had some users complaining that email verification was not working properly in some cases.
For example on ios, when a user clicks the verification link, link is opened in a new window without any sessions.
The auth middleware will now prevent the email verification process since the user is not logged in.

This minor fix ensures a redirect to the intended page after login. In this example, back to verification link after login, which works fine now.

Without this fix users would be redirected to the /verify-mail page after login (depending on the verified middleware), with the option to resend the verification mail. This process will then be repeated over and over.

This should also fix the situation registration on device A, validation on device B (which has no active session)

Redirect to intended path after login
@driesvints driesvints changed the title Redirect to intended path after login [1.x] Redirect to intended path after login Feb 11, 2021
@taylorotwell taylorotwell merged commit 2b1108a into laravel:1.x Feb 11, 2021
@martijnb92 martijnb92 deleted the patch-1 branch February 11, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants