[PR #2864] [CLOSED] validate billing_email on save #25381

New Issue

GiteaMirror · 2026-05-29T04:31:18-05:00

GiteaMirror commented

2026-05-29 04:31:18 -05:00

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/2864
Author: @stefan0xC
Created: 10/25/2022
Status: ❌ Closed

Base: main ← Head: validate-email

📝 Commits (1)

14bcbdc validate billing_email on save

📊 Changes

3 files changed (+9 additions, -0 deletions)

View changed files

📝 Cargo.lock (+4 -0)
📝 Cargo.toml (+1 -0)
📝 src/db/models/organization.rs (+4 -0)

📄 Description

As discussed in #2849 the billing email address should be validated server-side since the client does not currently check the field when updating it.

The same validation could be used for user.email but not sure if this is needed.

As far as I've tested it, this change should only affect creating and updating organizations as the organization itself is not saved in other functions but I might have missed something.

_{🔄 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/2864 **Author:** [@stefan0xC](https://github.com/stefan0xC) **Created:** 10/25/2022 **Status:** ❌ Closed **Base:** `main` ← **Head:** `validate-email` --- ### 📝 Commits (1) - [`14bcbdc`](https://github.com/dani-garcia/vaultwarden/commit/14bcbdcfbd4cc7bffd673790d68ef7f7a32f413d) validate billing_email on save ### 📊 Changes **3 files changed** (+9 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+4 -0) 📝 `Cargo.toml` (+1 -0) 📝 `src/db/models/organization.rs` (+4 -0) </details> ### 📄 Description As discussed in #2849 the billing email address should be validated server-side since the client does not currently check the field when updating it. The same validation could be used for user.email but not sure if this is needed. As far as I've tested it, this change should only affect creating and updating organizations as the organization itself is not saved in other functions but I might have missed something. --- <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-05-29 04:31:18 -05:00

GiteaMirror closed this issue

2026-05-29 04:31:19 -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#25381