Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sharing WRITE ACL do nothing more than READ #815

Closed
rmarso opened this issue Apr 11, 2024 · 2 comments · Fixed by #820
Closed

Sharing WRITE ACL do nothing more than READ #815

rmarso opened this issue Apr 11, 2024 · 2 comments · Fixed by #820
Assignees
Labels

Comments

@rmarso
Copy link

rmarso commented Apr 11, 2024

Hi,
First, Many Thanks for this App !!
Best regards.

Bug report

Sharing WRITE ACL do nothing more than READ

with READ+WRITE ACL => MODIFY is KO => button appears but when i click nothing appends
with READ+WRITE+FILE ACL => MODIFY is OK => button appears and when i click it opens the window

FILE ACL is mandatory to MODIFY a password or is it a bug ?

Steps to reproduce

  1. Login with account1 (admin group), create new entry and sharing with account2
  2. Login with account2 (admin group), create new entry and sharing with account1
  3. Sharing passwords between each other and tested on each side

Configuration

tested on fresh rocky 9 install with apache + php-fpm + mariadb
with nextcloud and passman in their last version
not tested with any previous
not tested with docker

@binsky08
Copy link
Collaborator

thanks for report @rmarso , I'll try to look into it soon

@binsky08
Copy link
Collaborator

It seems that the frontend tries to decrypt the files field even if the sharing permission for that is missing.
Positive: decryption of that field fails :)
Todo: it should not even try that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants