Keep warning level for server-side logging (#3263)

* Keep warning level for server-side logging * Fix tests
Kinto · Sep 13, 2023 · a3fc5ca · a3fc5ca
1 parent c14b5e7
commit a3fc5ca
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/kinto/core/authorization.py b/kinto/core/authorization.py
@@ -104,7 +104,7 @@ def permits(self, context, principals, permission):
                 allowed = context.check_permission(principals, bound_perms)
 
         if not allowed:
-            logger.warning(
+            logger.info(
                 "Permission %r on %r not granted to %r.",
                 permission,
                 object_id,

diff --git a/kinto/core/errors.py b/kinto/core/errors.py
@@ -205,7 +205,7 @@ def request_GET(request):
     except UnicodeDecodeError:
         querystring = request.environ.get("QUERY_STRING", "")
         logger = logging.getLogger(__name__)
-        logger.warning("Error decoding QUERY_STRING: %s" % request.environ)
+        logger.info("Error decoding QUERY_STRING: %s" % request.environ)
         raise http_error(
             httpexceptions.HTTPBadRequest(),
             errno=ERRORS.INVALID_PARAMETERS,

diff --git a/tests/core/test_authorization.py b/tests/core/test_authorization.py
@@ -185,7 +185,7 @@ def test_permits_logs_authz_failures(self):
         userid = "portier:yourself"
         object_id = "/articles/43/comments/2"
         perm = "read"
-        mocked.warning.assert_called_with(
+        mocked.info.assert_called_with(
             "Permission %r on %r not granted to %r.",
             perm,
             object_id,