[PR #3632] [MERGED] fix missing password check while manual reset password enrollment #8258

Closed
opened 2026-04-16 12:20:59 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/3632
Author: @sirux88
Created: 7/2/2023
Status: Merged
Merged: 7/4/2023
Merged by: @dani-garcia

Base: mainHead: fix-reset-password-check-issue


📝 Commits (2)

  • 9876aed added password check for manual reset
  • fb6f441 fixed unnecessary variable usage

📊 Changes

1 file changed (+12 additions, -0 deletions)

View changed files

📝 src/api/core/organizations.rs (+12 -0)

📄 Description

While enrolling reset password as user the web interface forces to type the password to authenticate oneself.
In its current state the backend doesn't check the password. Therefore a random passord can be used for authentification.

This PR solves this issue.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/dani-garcia/vaultwarden/pull/3632 **Author:** [@sirux88](https://github.com/sirux88) **Created:** 7/2/2023 **Status:** ✅ Merged **Merged:** 7/4/2023 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `fix-reset-password-check-issue` --- ### 📝 Commits (2) - [`9876aed`](https://github.com/dani-garcia/vaultwarden/commit/9876aedd6780c60c70c603e3bb329478bb0357ed) added password check for manual reset - [`fb6f441`](https://github.com/dani-garcia/vaultwarden/commit/fb6f441a4f15fed5cfeda12df437153017556844) fixed unnecessary variable usage ### 📊 Changes **1 file changed** (+12 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/organizations.rs` (+12 -0) </details> ### 📄 Description While enrolling reset password as user the web interface forces to type the password to authenticate oneself. In its current state the backend doesn't check the password. Therefore a random passord can be used for authentification. This PR solves this issue. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-16 12:20:59 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#8258