[PR #695] [MERGED] Stop leaking usernames when SIGNUPS_ALLOWED=false #2783

New issue

Closed

opened 2026-03-03 08:53:04 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 08:53:04 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/695
Author: @mprasil
Created: 11/1/2019
Status: ✅ Merged
Merged: 11/1/2019
Merged by: @dani-garcia

Base: master ← Head: do-not-leak-usernames

📝 Commits (1)

00a11b1 Stop leaking usernames when SIGNUPS_ALLOWED=false

📊 Changes

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

View changed files

📝 src/api/core/accounts.rs (+7 -3)

📄 Description

This fixes #691 - respond in less specific way to not leak the
fact that user is already registered on the server.

_{🔄 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/695 **Author:** [@mprasil](https://github.com/mprasil) **Created:** 11/1/2019 **Status:** ✅ Merged **Merged:** 11/1/2019 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `do-not-leak-usernames` --- ### 📝 Commits (1) - [`00a11b1`](https://github.com/dani-garcia/vaultwarden/commit/00a11b1b784af6283a8321d240a309bb637d23a6) Stop leaking usernames when SIGNUPS_ALLOWED=false ### 📊 Changes **1 file changed** (+7 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/accounts.rs` (+7 -3) </details> ### 📄 Description This fixes #691 - respond in less specific way to not leak the fact that user is already registered on the server. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>