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

missing permission bit for read-access #49

Open
moscicki opened this issue Oct 21, 2015 · 3 comments
Open

missing permission bit for read-access #49

moscicki opened this issue Oct 21, 2015 · 3 comments

Comments

@moscicki
Copy link
Contributor

@dragotin, @ckamm, @DeepDiver1975, @labkode:

There is a missing permission bit to represent Read access. Currently it is assumed that everything visible is readable which is not the case in general with external storage (home directories) where permissions may be handled outside of owncloud. So it could be useful for the sync client to discover if effectively a resource is readable or not (and for example, avoid recursing into such a directory which then would result in an error). You may consider if it would be useful to distinguish between ability to list files and read them (like AFS does it). I am not clear on it.

Ref:
https://github.com/owncloud/client/blob/master/doc/architecture.rst#server-side--permissions
https://github.com/cernbox/smashbox/blob/master/protocol/protocol.md#list-directory

@dragotin
Copy link

But something that is not readable, shouldn't it simply not be visible at all? Both in WebIF and WebDAV?

@labkode
Copy link
Member

labkode commented Oct 21, 2015

@moscicki @dragotin
I like the fact of having listing permissions and read/download permissions.
All storages I know from local filesystems (w bit on dir or r bit on file) to object storages (list a buck or read an object) make clear this separation, so this permission requirement could be managed at the storage level without having to use an external component (DB).

This concept could be extended also for share permissions to create new enterprise workflows. This is somehow related to owncloud/core#4579

This listing/read permissions are similar to the directory listing feature of Apache or Nginx.

@moscicki
Copy link
Contributor Author

@dragotin: suppose that you mount storage on external server. The read permission there may be removed for a file or subdirectory, yet the parent directory listing comes with this file listed. Doesn't it?

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

No branches or pull requests

3 participants