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

Error on setting a new password on recovery #8472

Closed
Smokyish opened this issue Oct 12, 2017 · 1 comment · Fixed by #8489
Closed

Error on setting a new password on recovery #8472

Smokyish opened this issue Oct 12, 2017 · 1 comment · Fixed by #8489

Comments

@Smokyish
Copy link

Smokyish commented Oct 12, 2017

Description:

A complex issue of the password recovery showing an Error when it should be working and showing an Error and still setting the password, but not informing the user of it.

Server Setup Information:

  • Version of Rocket.Chat Server: 0.58.4
  • Operating System: Linux
  • Deployment Method(snap/docker/tar/etc): docker
  • Number of Running Instances: 1
  • DB Replicaset Oplog:
  • Node Version: v4.8.4

Steps to Reproduce:

1. request password recovery
2. try to change password with being logged in as another user = error, no password change
3. try to change password without being logged = no error, password changed
no 2fa set

1. request password recovery
2. try to change password with being logged in as another user = error, no password change
3. try to change password without being logged in = error, no password change
4. request new password recovery
5. try to login without being logged in = error, password changed

Second was with 2fa, though i'm not convinced that it was a deciding factor

Expected behavior:

Password is changed without an error and user is informed that the password was successfully changed.

Actual behavior:

This seems like either a complex issue or multiple issues, finding consistency on the behaviour was difficult. Sometimes you get the error for no apparant reason, sometimes you get the error when you're logged in as another user and sometimes you get the error and the password is actually changed nonetheless.

Relevant logs:

Nothing in the logs with verbosity 0, Errors only.

@geekgonecrazy
Copy link
Member

@gdelavald not sure who should take a look. Please re-assign to appropriate person.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants