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

Reset password forget url #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Reset password forget url #3

wants to merge 1 commit into from

Conversation

Drivingralle
Copy link
Collaborator

Some plugins change the URL to the lost password interface.

WP-Default: https://www.example.com/wp-login.php?action=lostpassword
Woo-Default: https://www.example.com/my-account/lost-password/

If wp-logged-in-only and WooCommers are active a user can't reset the password without knowledge of the WP default URL. The Woo-Password-Reset-URL redirects the user to the frontend, but this is logged. wp-logged-in-only will then redirect the user back to the login form.
-> Loop

Therefore I propose this change so the password reset URL points to the default interface.

Maybe long term more filter need to be removed.

Some plugins change the URL to the lost password interface.

WP-Default: https://www.example.com/wp-login.php?action=lostpassword
Woo-Default: https://www.example.com/my-account/lost-password/

If wp-logged-in-only and WooCommers are active a user can't reset the password without knowledge of the WP default URL. The Woo-Password-Reset-URL redirects the user to the frontend, but this is logged. wp-logged-in-only will then redirect the user back to the login form.
-> Loop

Therefore I propose this change so the password reset URL points to the default interface.

Maybe long term more filter need to be removed.
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.

1 participant