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

New issue

Closed

opened 2026-03-03 09:41:49 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 09:41:49 +03:00

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