search

45Drives / cockpit-file-sharing

A Cockpit plugin to easily manage samba and NFS file sharing.
GNU General Public License v3.0
591 stars 32 forks source link

Newly created files and dirs cannot be modified when inherit owner = yes and acl_xattr:ignore system acls = yes. #117

Open it9exm opened 6 days ago

it9exm commented 6 days ago

Bug Info

Describe the bug Newly created files and dirs cannot be modified when inherit owner = yes and acl_xattr:ignore system acls = yes. If I select Windows ACLs with Linux/MacOS Support instead this does not occur, but ACLs will be created for every file/directory Is this normal?

To Reproduce Steps to reproduce the behavior: Create a share with 0770 permission and a bunch of users. owner/group are root:storage-users in my case. Set browseable, inherit permissions and windows acl toggles. Add "inherit owner = yes" in advanced settings. Also tried to add "acl group control = yes", without success. mount share and create files and dirs

Expected behavior I expect files and directories to have the same owner of the parent directory also with acl_xattr:ignore system acls = yes, and that such items can be manipulated.

Client Side

debian/sid, windows

Server Side

debian 12 lxc inside proxmox cockpit 325 samba 4.17

Additional context I just need to create a bucket-style shared storage space for a team of 3 people.

it9exm commented 5 days ago

fix/workaround: add the following in advanced acl_xattr:default acl style = everyone

it9exm commented 5 days ago

Taking advantage of this report for the following feature request: can acl_xattr:default acl style and inherit owner be made available as toggles or just integrated with the existing ones (Windows ACL)?