[PR #3168] [CLOSED] Increase secure notes length to match LastPass. #6881

New Issue

GiteaMirror · 2026-03-07T21:07:21-06:00

GiteaMirror commented

2026-03-07 21:07:21 -06:00

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/3168
Author: @STRML
Created: 1/25/2023
Status: ❌ Closed

Base: main ← Head: fix/secure-notes-length

📝 Commits (1)

15eeaee Increase secure notes length to match LastPass.

📊 Changes

2 files changed (+4 additions, -4 deletions)

View changed files

📝 src/api/core/ciphers.rs (+2 -2)
📝 src/db/models/cipher.rs (+2 -2)

📄 Description

As users are migrating from LastPass, I am seeing more and more people hitting the max length of 10,000 on the "Notes" field, which applies to both CipherType.Login and CipherType.SecureNote.

This default is too low, and by being significantly lower than competing products, needlessly complicates the transition.

This is not helped by how unhelpful the import process is on web (showing errors in console only, silently dropping items), which is a PR for another day. By merging this, we can ensure that one of the most common drop-offs is fixed.

45,000 matches LastPass. As we know that the encrypted length can be longer than the raw text, I have increased this to 50,000 so there is a healthy buffer to account for this inflation.

Mirrors PR at https://github.com/bitwarden/server/pull/2625

_{🔄 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/3168 **Author:** [@STRML](https://github.com/STRML) **Created:** 1/25/2023 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/secure-notes-length` --- ### 📝 Commits (1) - [`15eeaee`](https://github.com/dani-garcia/vaultwarden/commit/15eeaee6bc9bdb781dff33ac0d5f14aa9de2250c) Increase secure notes length to match LastPass. ### 📊 Changes **2 files changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/ciphers.rs` (+2 -2) 📝 `src/db/models/cipher.rs` (+2 -2) </details> ### 📄 Description As users are migrating from LastPass, I am seeing more and more people hitting the max length of 10,000 on the "Notes" field, which applies to both CipherType.Login and CipherType.SecureNote. This default is too low, and by being significantly lower than competing products, needlessly complicates the transition. This is not helped by how unhelpful the import process is on web (showing errors in console only, silently dropping items), which is a PR for another day. By merging this, we can ensure that one of the most common drop-offs is fixed. 45,000 matches LastPass. As we know that the encrypted length can be longer than the raw text, I have increased this to 50,000 so there is a healthy buffer to account for this inflation. Mirrors PR at https://github.com/bitwarden/server/pull/2625 --- <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-03-07 21:07:21 -06:00

GiteaMirror closed this issue

2026-03-07 21:07:22 -06:00

GiteaMirror referenced this issue

2026-03-07 21:18:31 -06:00

[PR #6881] [CLOSED] Add Google S2 icon cache fallback for favicon fetching #7497

GiteaMirror referenced this issue

2026-04-20 15:57:13 -05:00

[PR #6881] [CLOSED] Add Google S2 icon cache fallback for favicon fetching #12723

GiteaMirror referenced this issue

2026-04-23 08:03:06 -05:00

[PR #6881] [CLOSED] Add Google S2 icon cache fallback for favicon fetching #16703