Don't show Organization ciphers to unconfirmed users #41

Closed
opened 2026-02-04 16:22:37 +03:00 by OVERLORD · 5 comments
Owner

Originally created by @mprasil on GitHub (Jul 13, 2018).

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

Originally created by @mprasil on GitHub (Jul 13, 2018). This leads to confusion and ciphers that can't be decrypted (#77)
OVERLORD added the bug label 2026-02-04 16:22:37 +03:00
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.

@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.

@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?

@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.

@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.

@mprasil commented on GitHub (Jul 17, 2018): This should be completely resolved with #89 merged in.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#41