[PR #460] [MERGED] Fixed purging organization vault #3711

Closed
opened 2025-10-09 18:28:36 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/460
Author: @janost
Created: 4/14/2019
Status: Merged
Merged: 4/14/2019
Merged by: @dani-garcia

Base: masterHead: organization-vault-purge


📝 Commits (1)

  • 03fdf36 Fixed purging organization vault

📊 Changes

1 file changed (+40 additions, -13 deletions)

View changed files

📝 src/api/core/ciphers.rs (+40 -13)

📄 Description

Fixes #458.

The new logic checks for the organizationId query parameter.
If there is none, it assumes the users wants to purge their own vault.
If there is, it only purges the vault of the organization if the user is an owner of the organization.


🔄 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/460 **Author:** [@janost](https://github.com/janost) **Created:** 4/14/2019 **Status:** ✅ Merged **Merged:** 4/14/2019 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `organization-vault-purge` --- ### 📝 Commits (1) - [`03fdf36`](https://github.com/dani-garcia/vaultwarden/commit/03fdf36bf9f63068da5ecddbe9ea98d512fd2259) Fixed purging organization vault ### 📊 Changes **1 file changed** (+40 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/ciphers.rs` (+40 -13) </details> ### 📄 Description Fixes #458. The new logic checks for the `organizationId` query parameter. If there is none, it assumes the users wants to purge their own vault. If there is, it only purges the vault of the organization if the user is an owner of the organization. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2025-10-09 18:28:36 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#3711