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

Password reset link incorrect #429

Open
auggillet opened this issue Jan 23, 2023 · 0 comments
Open

Password reset link incorrect #429

auggillet opened this issue Jan 23, 2023 · 0 comments
Labels

Comments

@auggillet
Copy link

IMPORTANT: See the Troubleshooting page: https://docs.bracketspace.com/notification/user-guide/troubleshooting before creating an issue.

Bug description

When using notification plugin to send password reset emails, the mail is correctly sent to the user, but if the user ID is an email adress, the link generated through the plugin doesnt work : indeed, the "@" (or %40) character which is in the URL does not appear. So when the user clicks on the link, is doesnt work as the login is incorrect

To reproduce

Steps to reproduce the behavior:

  1. Setup the password reset trigger through email
  2. Create a wordpress user which login is an email (must contain "@")
  3. Send a password reset to this user
  4. The mail is correctly sent, but the link in it doesnt work as the URL does not contain a proper login

Expected behavior

The password reset link should contain "%40" or "@" in order to work

Environment

  • PHP: 7.4.32
  • WordPress: 6.1.1
  • Notification plugin: 8.0.14

Screenshots

image
image

@auggillet auggillet added the bug label Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant