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

More permissions control #134

Closed
edudele opened this issue Mar 23, 2017 · 3 comments
Closed

More permissions control #134

edudele opened this issue Mar 23, 2017 · 3 comments
Assignees
Milestone

Comments

@edudele
Copy link

edudele commented Mar 23, 2017

If you removed all capabilities, you can still create folder.
It is a permit that is not contemplated.

If all capabilities are considered. All permissions should be set:

if(!$this->hasPermission('addfolder')) { $this->error(sprintf($this->lang('NOT_ALLOWED'))); }
protected $actions_list = ["select", "upload", "download", "rename", "copy", "move", "replace", "delete", "edit", "addfolder"];

Thanks

@psolom
Copy link
Owner

psolom commented Mar 23, 2017

Thanks for reporting. According to the #111 it's planned to remove $actions_list completely due to security reasons. There will only two modes at the server-side: "read" and "write". If you have write permission you will be able to create folders, otherwise not.

@edudele
Copy link
Author

edudele commented Mar 23, 2017

It is much easier to have a writing profile and a reading profile to avoid all these problems. Thank you

@psolom psolom added this to the 2.4.0 milestone Apr 1, 2017
@psolom psolom self-assigned this May 6, 2017
@psolom
Copy link
Owner

psolom commented May 7, 2017

New backend security model is released in v2.4.0. The $actions_list is completely removed, you have to use readOnly server-side configuration option: https://github.com/servocoder/RichFilemanager-PHP#entry-point-setup

@psolom psolom closed this as completed May 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants