[PR #2945] [MERGED] Limit Cipher Note encrypted string size #3056

Closed
opened 2025-11-07 08:02:02 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/2945
Author: @BlackDex
Created: 12/2/2022
Status: Merged
Merged: 12/4/2022
Merged by: @dani-garcia

Base: mainHead: issue-2937


📝 Commits (1)

  • 7f11363 Limit Cipher Note encrypted string size

📊 Changes

1 file changed (+6 additions, -0 deletions)

View changed files

📝 src/api/core/ciphers.rs (+6 -0)

📄 Description

As discussed in #2937, this will limit the amount of encrypted characters to 10.000 characters, same as Bitwarden. This will not break current ciphers which exceed this limit, but it will prevent those ciphers from being updated.

Fixes #2937


🔄 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/2945 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 12/2/2022 **Status:** ✅ Merged **Merged:** 12/4/2022 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `issue-2937` --- ### 📝 Commits (1) - [`7f11363`](https://github.com/dani-garcia/vaultwarden/commit/7f11363725f48868eafc1c81a92e89f31da08659) Limit Cipher Note encrypted string size ### 📊 Changes **1 file changed** (+6 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/ciphers.rs` (+6 -0) </details> ### 📄 Description As discussed in #2937, this will limit the amount of encrypted characters to 10.000 characters, same as Bitwarden. This will not break current ciphers which exceed this limit, but it will prevent those ciphers from being updated. Fixes #2937 --- <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 2025-11-07 08:02:02 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#3056