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 and permissions #2658

Closed
Dreadeath opened this issue Dec 13, 2016 · 3 comments
Closed

Sharing and permissions #2658

Dreadeath opened this issue Dec 13, 2016 · 3 comments
Assignees
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: sharing

Comments

@Dreadeath
Copy link

Dreadeath commented Dec 13, 2016

Steps to reproduce

  1. User A creates folder and share it with User B without deleting permissions
  2. User B uploads file to sharing folder

Expected behaviour

User A should be able to delete uploaded file in folder which he owns and shares with User B

Actual behaviour

User A cannot delete uploaded file, like sharing permissions on User B also affects User A. In filecache table I can see limited permissions column parameters. This behavior could be "fixed" with occ files:scan User A command - after that User A can delete file, User B can't, permissions in filecache table transfers to expected.

Server configuration

Operating system:
Linux
Web server:
nginx 1.10.2
Database:
Mariadb 10.1.19
PHP version:
7.0.14
Nextcloud version: (see Nextcloud admin page)
11.0.0, previous 10.something was affected too
Updated from an older Nextcloud/ownCloud or fresh install:
from Nextcloud 10.something, from owncloud 9.something
Where did you install Nextcloud from:
AUR
Signing status:

Signing status No errors have been found.

List of activated apps:

App list Enabled: - activity: 2.4.1 - admin_audit: 1.1.0 - bookmarks: 0.9.1 - calendar: 1.4.1 - comments: 1.1.0 - contacts: 1.5.2 - dav: 1.1.1 - direct_menu: 0.9.3 - encryption: 1.4.1 - federatedfilesharing: 1.1.1 - federation: 1.1.1 - files: 1.6.1 - files_automatedtagging: 1.1.1 - files_external: 1.1.2 - files_pdfviewer: 1.0.1 - files_sharing: 1.1.1 - files_texteditor: 2.2 - files_trashbin: 1.1.0 - files_versions: 1.4.0 - files_videoplayer: 1.0.0 - gallery: 16.0.0 - logreader: 2.0.0 - lookup_server_connector: 1.0.0 - mail: 0.6.2 - nextcloud_announcements: 1.0 - notes: 2.1.0 - notifications: 1.0.1 - password_policy: 1.1.0 - provisioning_api: 1.1.0 - serverinfo: 1.1.1 - sharebymail: 1.0.1 - survey_client: 0.1.5 - systemtags: 1.1.3 - theming: 1.1.1 - twofactor_backupcodes: 1.0.0 - updatenotification: 1.1.1 - workflowengine: 1.1.1 Disabled: - external - files_accesscontrol - files_retention - firstrunwizard - tasks - templateeditor - user_external - user_ldap - user_saml

Are you using external storage, if yes which one: local/smb/sftp/...
no
Are you using encryption: yes/no
no
Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...
no

Logs

Web server error log

Web server error log no logs

Nextcloud log (data/nextcloud.log)

Nextcloud log no logs
@LukasReschke
Copy link
Member

cc @nextcloud/sharing

@LukasReschke LukasReschke added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Dec 13, 2016
@LukasReschke LukasReschke added this to the Nextcloud 11.0.1 milestone Dec 13, 2016
@rullzer
Copy link
Member

rullzer commented Dec 13, 2016

I can verify this.
It is because we mask the storage. @icewind1991 please take a look

@LukasReschke
Copy link
Member

Also an integration test won't hurt here. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: sharing
Projects
None yet
Development

No branches or pull requests

4 participants