You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Having just played with this code, I'd like to suggest a change to how default acls are handled. Technically, the ACL and Default ACL for a directory are two different resources, so I think something like this would be better:
acl /tmp/directory do
user 'rwx'
group '': 'r-x', testgroup: 'rwx'
mask 'rwx'
end
default_acl /tmp/directory do
user 'rwx'
group '': 'r-x', testgroup: 'rwx'
mask 'rwx'
end
That would also allow for having an action :remove to remove acls or default acls individually.
I might be able to do the work, and would also recommend being able to accept the current form with a deprecation warning. Didn't want to do the work and submit a PR without having a design discussion though.
The text was updated successfully, but these errors were encountered:
Having just played with this code, I'd like to suggest a change to how default acls are handled. Technically, the ACL and Default ACL for a directory are two different resources, so I think something like this would be better:
That would also allow for having an
action :remove
to remove acls or default acls individually.I might be able to do the work, and would also recommend being able to accept the current form with a deprecation warning. Didn't want to do the work and submit a PR without having a design discussion though.
The text was updated successfully, but these errors were encountered: