[PR #5691] [MERGED] fix invitation logic for new registration flow #12509

Closed
opened 2026-04-20 15:47:35 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/5691
Author: @stefan0xC
Created: 3/14/2025
Status: Merged
Merged: 3/15/2025
Merged by: @dani-garcia

Base: register_verify_emailHead: register_verify_email


📝 Commits (2)

  • 1d5606f fix invitation logic for new registration flow
  • a13f20a clarify email_2fa_enforce_on_verified_invite

📊 Changes

3 files changed (+12 additions, -18 deletions)

View changed files

📝 .env.template (+1 -1)
📝 src/api/core/accounts.rs (+10 -16)
📝 src/config.rs (+1 -1)

📄 Description

My suggestions for your open pull request #5215 which should be backwards compatible with the old registration flow as well as make it possible to enforce email 2fa for all users regardless of any organization policy as discussed here https://github.com/dani-garcia/vaultwarden/discussions/5662#discussioncomment-12398665.


🔄 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/5691 **Author:** [@stefan0xC](https://github.com/stefan0xC) **Created:** 3/14/2025 **Status:** ✅ Merged **Merged:** 3/15/2025 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `register_verify_email` ← **Head:** `register_verify_email` --- ### 📝 Commits (2) - [`1d5606f`](https://github.com/dani-garcia/vaultwarden/commit/1d5606ffe219a3194bbf8417c17bd3249f6a361c) fix invitation logic for new registration flow - [`a13f20a`](https://github.com/dani-garcia/vaultwarden/commit/a13f20a63f63044c72dd12e328fe7f4614307f40) clarify email_2fa_enforce_on_verified_invite ### 📊 Changes **3 files changed** (+12 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `.env.template` (+1 -1) 📝 `src/api/core/accounts.rs` (+10 -16) 📝 `src/config.rs` (+1 -1) </details> ### 📄 Description My suggestions for your open pull request #5215 which should be backwards compatible with the old registration flow as well as make it possible to enforce email 2fa for all users regardless of any organization policy as discussed here https://github.com/dani-garcia/vaultwarden/discussions/5662#discussioncomment-12398665. --- <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-20 15:47:36 -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#12509