[GH-ISSUE #80] Don't show Organization ciphers to unconfirmed users #41

Closed
opened 2026-03-03 01:24:05 +03:00 by kerem · 5 comments
Owner

Originally created by @mprasil on GitHub (Jul 13, 2018).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/80

This leads to confusion and ciphers that can't be decrypted (#77)

Originally created by @mprasil on GitHub (Jul 13, 2018). Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/80 This leads to confusion and ciphers that can't be decrypted (#77)
kerem 2026-03-03 01:24:05 +03:00
  • closed this issue
  • added the
    bug
    label
Author
Owner

@mprasil commented on GitHub (Jul 13, 2018):

I've totally missed the 2f892cb commit. So chances are this might be resolved already. Going to rebuild the image and test.

<!-- gh-comment-id:404775076 --> @mprasil commented on GitHub (Jul 13, 2018): I've totally missed the 2f892cb commit. So chances are this might be resolved already. Going to rebuild the image and test.
Author
Owner

@dani-garcia commented on GitHub (Jul 13, 2018):

It's a partial fix, at least. This hides the ciphers, but we should probably also not show the organization or the collections unless the user is confirmed.

We should also make sure that an unconfirmed admin or owner can't make changes to the org.

<!-- gh-comment-id:404775413 --> @dani-garcia commented on GitHub (Jul 13, 2018): It's a partial fix, at least. This hides the ciphers, but we should probably also not show the organization or the collections unless the user is confirmed. We should also make sure that an unconfirmed admin or owner can't make changes to the org.
Author
Owner

@mprasil commented on GitHub (Jul 13, 2018):

Good point do you want to keep the issue open for that?

<!-- gh-comment-id:404777402 --> @mprasil commented on GitHub (Jul 13, 2018): Good point do you want to keep the issue open for that?
Author
Owner

@dani-garcia commented on GitHub (Jul 13, 2018):

Yeah, I think we should leave it open for now.

<!-- gh-comment-id:404781315 --> @dani-garcia commented on GitHub (Jul 13, 2018): Yeah, I think we should leave it open for now.
Author
Owner

@mprasil commented on GitHub (Jul 17, 2018):

This should be completely resolved with #89 merged in.

<!-- gh-comment-id:405527843 --> @mprasil commented on GitHub (Jul 17, 2018): This should be completely resolved with #89 merged in.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vaultwarden#41
No description provided.