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

User logged out when clicking on setpassword link in emails with newly created accounts. #1422

Closed
angrybrad opened this issue Feb 24, 2017 · 0 comments
Assignees
Labels
bug severity:minor Cosmetic issues or bugs with simple workarounds

Comments

@angrybrad
Copy link
Member

Description

When a new user gets the activate your account email with the setpassword link in it, they can sometimes double-click on the hyperlink.

The first click silently activates the account and the second click tries to validate immediately after and an error is thrown, sending folks off to the activateAccountFailurePath config setting and leading to lots of end user confusion.

This happens because UsersController::_processTokenRequest() (which is used by both actionSetPassword and actionVerifyPassword), will immediately check to see if there is a logged in user and if so, log them out first.

Similar to #1421

Steps to reproduce

  1. Register a new user.
  2. Double click the hyperlink in the email.

Additional info

  • Craft version: 2.x and 3.x
  • PHP version: n/a
  • Database driver & version: n/a
  • Plugins & versions: n/a
@angrybrad angrybrad added bug severity:minor Cosmetic issues or bugs with simple workarounds labels Feb 24, 2017
@brandonkelly brandonkelly self-assigned this Mar 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug severity:minor Cosmetic issues or bugs with simple workarounds
Projects
None yet
Development

No branches or pull requests

2 participants