[PR #1921] [MERGED] Fix wrong display of MFA email. #11889

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

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/1921
Author: @BlackDex
Created: 8/18/2021
Status: Merged
Merged: 8/22/2021
Merged by: @dani-garcia

Base: mainHead: fix-1878


📝 Commits (1)

  • 881d1f4 Fix wrong display of MFA email.

📊 Changes

1 file changed (+7 additions, -5 deletions)

View changed files

📝 src/api/core/two_factor/email.rs (+7 -5)

📄 Description

There was some wrong logic regarding the display of which email is
configured to be used for the email MFA. This is now fixed.

Resolves #1878


🔄 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/1921 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 8/18/2021 **Status:** ✅ Merged **Merged:** 8/22/2021 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `fix-1878` --- ### 📝 Commits (1) - [`881d1f4`](https://github.com/dani-garcia/vaultwarden/commit/881d1f43340b16cfa6b9836b520537461397c1d3) Fix wrong display of MFA email. ### 📊 Changes **1 file changed** (+7 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/two_factor/email.rs` (+7 -5) </details> ### 📄 Description There was some wrong logic regarding the display of which email is configured to be used for the email MFA. This is now fixed. Resolves #1878 --- <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:22:18 -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#11889