Avii/Radicale
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

sharing: adjust loglevel or add forgotten log

Browse Source
This commit is contained in:
Peter Bieringer
2026-04-21 18:55:14 +02:00
parent 53d5a89165
commit eecfdcaf69
3 changed files with 14 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
radicale/app/__init__.py
View File

@@ -614,9 +614,9 @@ class Application(ApplicationPartDelete, ApplicationPartHead,
if (status, headers, answer, xml_request) == httputils.NOT_ALLOWED: if (status, headers, answer, xml_request) == httputils.NOT_ALLOWED:
if path.startswith("/.token"): if path.startswith("/.token"):
logger.info("Access to %r denied", path) logger.notice("Access to %r denied", path)
else: else:
logger.info("Access to %r denied for %s", path, repr(user) if user else "anonymous user") logger.notice("Access to %r denied for %s", path, repr(user) if user else "anonymous user")
else: else:
status, headers, answer, xml_request = httputils.NOT_ALLOWED status, headers, answer, xml_request = httputils.NOT_ALLOWED

10
radicale/app/proppatch.py
View File

@@ -127,13 +127,13 @@ class ApplicationPartProppatch(ApplicationBase):
(not 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.notice("PROPPATCH request on shared %r: write-access, overlay enforced, but disabled by share permission 'e'", path_orig)
elif "e" in raw_permissions: elif "e" in raw_permissions:
logger.info("PROPPATCH request on shared %r: write-access, overlay enforced, but disabled by rights permission 'e'", path_orig) logger.notice("PROPPATCH request on shared %r: write-access, overlay enforced, but disabled by rights permission 'e'", path_orig)
else: else:
share_overlay = True share_overlay = True
else: else:
logger.info("PROPPATCH request on shared %r: no write-access", path_orig) logger.notice("PROPPATCH request on shared %r: no write-access", path_orig)
return httputils.NOT_ALLOWED return httputils.NOT_ALLOWED
else: else:
return httputils.NOT_ALLOWED return httputils.NOT_ALLOWED
@@ -144,9 +144,9 @@ class ApplicationPartProppatch(ApplicationBase):
logger.trace("PROPPATCH/xml_proppatch: write-access/sharing: %r", path_orig) logger.trace("PROPPATCH/xml_proppatch: write-access/sharing: %r", path_orig)
if self._sharing.enforce_properties_overlay: if self._sharing.enforce_properties_overlay:
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-permissions, overlay enforced, but disabled by share permission 'e'", path_orig) logger.notice("PROPPATCH request on shared %r: write-permissions, overlay enforced, but disabled by share permission 'e'", path_orig)
elif "e" in raw_permissions: elif "e" in raw_permissions:
logger.info("PROPPATCH request on shared %r: write-permissions, overlay enforced, but disabled by rights permission 'e'", path_orig) logger.notice("PROPPATCH request on shared %r: write-permissions, overlay enforced, but disabled by rights permission 'e'", path_orig)
else: else:
share_overlay = True share_overlay = True
else: else:

8
radicale/sharing/__init__.py
View File

@@ -1040,7 +1040,7 @@ class BaseSharing(ApplicationBase):
else: else:
answer['PathOrToken'] = token answer['PathOrToken'] = token
logger.info(api_info + " success: PathMapped=%r Permissions=%r PathOrToken=%r", PathMapped, Permissions, PathOrToken) logger.notice(api_info + " success: PathMapped=%r Permissions=%r PathOrToken=%r", PathMapped, Permissions, PathOrToken)
# action: update # action: update
elif action == "update": elif action == "update":
@@ -1172,6 +1172,8 @@ class BaseSharing(ApplicationBase):
logger.warning(api_info + ": %r not successful", request_data['PathOrToken']) logger.warning(api_info + ": %r not successful", request_data['PathOrToken'])
return httputils.bad_request("Internal Error") return httputils.bad_request("Internal Error")
logger.notice(api_info + " success: PathMapped=%r PathOrToken=%r", PathMapped, PathOrToken)
# action: delete # action: delete
elif action == "delete": elif action == "delete":
logger.trace("" + api_info + ": start") logger.trace("" + api_info + ": start")
@@ -1212,6 +1214,8 @@ class BaseSharing(ApplicationBase):
logger.warning(api_info + ": %r by user %r not successful", request_data['PathOrToken'], request_data['User']) logger.warning(api_info + ": %r by user %r not successful", request_data['PathOrToken'], request_data['User'])
return httputils.bad_request("Internal Error") return httputils.bad_request("Internal Error")
logger.notice(api_info + " success: PathMapped=%r PathOrToken=%r", PathMapped, PathOrToken)
# action: info # action: info
elif action == "info": elif action == "info":
logger.info(api_info + ": success") logger.info(api_info + ": success")
@@ -1300,6 +1304,8 @@ class BaseSharing(ApplicationBase):
logger.warning(api_info + ": %r by user %s not successful", request_data['PathOrToken'], user) logger.warning(api_info + ": %r by user %s not successful", request_data['PathOrToken'], user)
return httputils.bad_request("Internal Error") return httputils.bad_request("Internal Error")
logger.notice(api_info + " success: PathMapped=%r PathOrToken=%r", PathMapped, PathOrToken)
else: else:
# default # default
logger.warning(api_info + ": unsupported action=%r", action) logger.warning(api_info + ": unsupported action=%r", action)