mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-07-17 01:12:27 -05:00
[GH-ISSUE #885] Favorites are tracked at cipher level rather than on a per-user basis #28119
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jjlin on GitHub (Mar 1, 2020).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/885
Currently, whether a cipher is a "favorite" is a property of the cipher itself, but it should probably be tracked similarly to folders, with a separate "favorites" table that maps users to ciphers. Otherwise, for a shared cipher, when one user sets a cipher as a "favorite", it automatically becomes every other user's favorite too, which probably isn't the intended behavior.
Also, when changing the "favorite" property, the cipher update time probably shouldn't be updated (similar to #884).