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

New issue

Closed

opened 2026-03-03 09:25:28 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 09:25:28 +03:00

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: main ← Head: 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>