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.

Sign up for GitHub

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

Jump to bottom

Reset Password Fails on Second Attempt #452

Closed
michaelquigley opened this issue Nov 14, 2023 · 1 comment
Closed

Reset Password Fails on Second Attempt #452

michaelquigley opened this issue Nov 14, 2023 · 1 comment
Assignees
@camotts
Labels
defect Something isn't working
Milestone
v0.4

Comments

@michaelquigley
Copy link
Collaborator

Second attempts at password resets update the previous deleted request record, instead of creating a new record.

Probably has been broken since the soft deletes schema change.
@michaelquigley michaelquigley added the defect Something isn't working label Nov 14, 2023
@michaelquigley michaelquigley added this to the v0.4 milestone Nov 14, 2023
@michaelquigley
Copy link
Collaborator Author

Looks like this should just be a matter of dropping the ON CONFLICT portion of the INSERT statement in controller/store/password_reset_request.go (CreatePasswordResetRequest).

michaelquigley added a commit that referenced this issue Feb 15, 2024
@michaelquigley
Merge pull request #542 from openziti/password-reset-fix
61101bd 
Fix for multiple password reset requests per account (#452)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@camotts camotts

Labels
defect Something isn't working
Projects
zrok Development Roadmap
Status: Done
Milestone
v0.4
Development

No branches or pull requests
2 participants
@michaelquigley @camotts