sharing/propfind+proppatch: fix permission check for properties overlay
This commit is contained in:
@@ -313,7 +313,7 @@ def xml_propfind_response(
|
|||||||
) and not (
|
) and not (
|
||||||
"p" in share['Permissions'] or
|
"p" in share['Permissions'] or
|
||||||
("p" in raw_permissions and "P" not in share['Permissions']) or
|
("p" in raw_permissions and "P" not in share['Permissions']) or
|
||||||
(self._sharing.permit_properties_overlay and "P" not in raw_permissions and "P" not in share['Permissions'])):
|
(not self._sharing.permit_properties_overlay and "P" not in raw_permissions and "P" not in share['Permissions'])):
|
||||||
logger.trace("PROPFIND/xml_propfind_response/current-user-privilege-set: add D:write-properties")
|
logger.trace("PROPFIND/xml_propfind_response/current-user-privilege-set: add D:write-properties")
|
||||||
privileges.append("D:write-properties")
|
privileges.append("D:write-properties")
|
||||||
elif write:
|
elif write:
|
||||||
|
|||||||
@@ -124,7 +124,7 @@ class ApplicationPartProppatch(ApplicationBase):
|
|||||||
) and not (
|
) and not (
|
||||||
"p" in share['Permissions'] or
|
"p" in share['Permissions'] or
|
||||||
("p" in raw_permissions and "P" not in share['Permissions']) or
|
("p" in raw_permissions and "P" not in share['Permissions']) or
|
||||||
(self._sharing.permit_properties_overlay and "P" not in raw_permissions and "P" not in share['Permissions'])):
|
(not self._sharing.permit_properties_overlay and "P" not in raw_permissions and "P" not in share['Permissions'])):
|
||||||
logger.info("PROPPATCH request on shared %r: write-access", path_orig)
|
logger.info("PROPPATCH request on shared %r: write-access", path_orig)
|
||||||
if permissions_filter is not None and "e" in permissions_filter:
|
if permissions_filter is not None and "e" in permissions_filter:
|
||||||
logger.info("PROPPATCH request on shared %r: write-access, overlay enforced, but disabled by share permission 'e'", path_orig)
|
logger.info("PROPPATCH request on shared %r: write-access, overlay enforced, but disabled by share permission 'e'", path_orig)
|
||||||
|
|||||||
Reference in New Issue
Block a user