Fixing permissions access #215
Conversation
| { type: "divider" }, | ||
| { title: cockpit.format(_("Delete")), onClick: _deleteItem, className: "pf-m-danger" }, | ||
| ]) | ||
| : superuser.allowed |
There was a problem hiding this comment.
This doesn't work, this needs to deal with UNIX permissions from the selected file. More details for example here.
There was a problem hiding this comment.
What exactly doesn't work? Did I misunderstand the issue or implemented it wrongly?
I hid create link while in "limited access" mode
There was a problem hiding this comment.
The code needs to look at the current file permissions and the current logged in user, not if they are sudo or not.
I can only recommend researching how UNIX file permissions work in general https://linuxize.com/post/understanding-linux-file-permissions/
There was a problem hiding this comment.
Aha So every file should be worked on independently?
There was a problem hiding this comment.
Yes, every file needs to be checked independent.
|
This has gone out of sync for a while now, closing. |
This PR aims to fix #178