starred/vaultwarden

Fork 0

mirror of https://github.com/dani-garcia/vaultwarden.git synced 2026-04-25 17:25:57 +03:00

[GH-ISSUE #383] Error saving cipher #222

New issue

Closed

opened 2026-03-03 01:26:53 +03:00 by kerem · 7 comments

kerem commented

2026-03-03 01:26:53 +03:00

Owner

Originally created by @romain1206 on GitHub (Feb 4, 2019).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/383

[2019-02-04 22:45:47][bitwarden_rs::db::models::user][WARN] Failed to update revision for xxxxxxxx@xxxx.fr
[2019-02-04 22:45:47][bitwarden_rs::error][ERROR] Error saving cipher.
[CAUSE] DatabaseError(
__Unknown,
"database is locked"
)

Originally created by @romain1206 on GitHub (Feb 4, 2019). Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/383 [2019-02-04 22:45:47][bitwarden_rs::db::models::user][WARN] Failed to update revision for xxxxxxxx@xxxx.fr [2019-02-04 22:45:47][bitwarden_rs::error][ERROR] Error saving cipher. [CAUSE] DatabaseError( __Unknown, "database is locked" )

kerem closed this issue

2026-03-03 01:26:53 +03:00

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@dani-garcia commented on GitHub (Feb 4, 2019):

Can you give us some steps to reproduce the error?

@dani-garcia commented on GitHub (Feb 4, 2019): Can you give us some steps to reproduce the error?

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@romain1206 commented on GitHub (Feb 5, 2019):

hello,

i have this error when i want move 300 login to another folder.

@romain1206 commented on GitHub (Feb 5, 2019): hello, i have this error when i want move 300 login to another folder.

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@mprasil commented on GitHub (Feb 5, 2019):

We do a bunch of updates there in the DB and we also trigger a bunch of update_revision calls (one for each move), maybe we should only trigger one at the end of the operation.

@mprasil commented on GitHub (Feb 5, 2019): We do a bunch of updates there in the DB and we also trigger a bunch of update_revision calls (one for each move), maybe we should only trigger one at the end of the operation.

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@mprasil commented on GitHub (Feb 5, 2019):

I was thinking that maybe we could defer the revision update and try to compact multiple revision updates into one, but some of the vault functionality actually relies on revision being updated instantly.

I think we can attempt to retry when updating the revision rather than failing straight away. I'll try to implement that.

@mprasil commented on GitHub (Feb 5, 2019): I was thinking that maybe we could defer the revision update and try to compact multiple revision updates into one, but some of the vault functionality actually relies on revision being updated instantly. I think we can attempt to retry when updating the revision rather than failing straight away. I'll try to implement that.

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@mprasil commented on GitHub (Feb 5, 2019):

Okay, I've created a PR, that should hopefully fix this. The revision update should now retry 10 times instead of failing instantly. (similar to attachment updates) I've tried to move about 500 items (as much as Vault let me) and it seems to handle that fine.

@mprasil commented on GitHub (Feb 5, 2019): Okay, I've created a PR, that should hopefully fix this. The revision update should now retry 10 times instead of failing instantly. (similar to attachment updates) I've tried to move about 500 items (as much as Vault let me) and it seems to handle that fine.

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@dani-garcia commented on GitHub (Feb 5, 2019):

This hopefully has been fixed with the latest PR. We'll have to wait an hour or so until the docker images are built.

@dani-garcia commented on GitHub (Feb 5, 2019): This hopefully has been fixed with the latest PR. We'll have to wait an hour or so until the docker images are built.

kerem commented

2026-03-03 01:26:54 +03:00

Author

Owner

@suricloud commented on GitHub (Mar 18, 2021):

Hi,just installed,and getting this same error?

@suricloud commented on GitHub (Mar 18, 2021): Hi,just installed,and getting this same error?

kerem referenced this issue

2026-03-03 08:36:27 +03:00

[PR #225] [MERGED] Don't send Domains if excludeDomains=true on /sync #2675