Log handled errors to warning. Closes #1925.

spec-first · May 14, 2024 · 9a10f29 · 9a10f29
1 parent fcfe151
commit 9a10f29
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/connexion/middleware/exceptions.py b/connexion/middleware/exceptions.py
@@ -72,7 +72,12 @@ def add_exception_handler(
     @staticmethod
     def problem_handler(_request: ConnexionRequest, exc: ProblemException):
         """Default handler for Connexion ProblemExceptions"""
-        logger.error("%r", exc)
+
+        if 400 <= exc.status <= 499:
+            logger.warning("%r", exc)
+        else:
+            logger.error("%r", exc)
+
         return exc.to_problem()
 
     @staticmethod
@@ -81,6 +86,12 @@ def http_exception(
         _request: StarletteRequest, exc: HTTPException, **kwargs
     ) -> StarletteResponse:
         """Default handler for Starlette HTTPException"""
+
+        if 400 <= exc.status_code <= 499:
+            logger.warning("%r", exc)
+        else:
+            logger.error("%r", exc)
+
         logger.error("%r", exc)
         return problem(
             title=http_facts.HTTP_STATUS_CODES.get(exc.status_code),

diff --git a/connexion/validators/json.py b/connexion/validators/json.py
@@ -68,7 +68,7 @@ def _validate(self, body: t.Any) -> t.Optional[dict]:
             return self._validator.validate(body)
         except ValidationError as exception:
             error_path_msg = format_error_with_path(exception=exception)
-            logger.error(
+            logger.info(
                 f"Validation error: {exception.message}{error_path_msg}",
                 extra={"validator": "body"},
             )
@@ -129,7 +129,7 @@ def _validate(self, body: dict):
             self.validator.validate(body)
         except ValidationError as exception:
             error_path_msg = format_error_with_path(exception=exception)
-            logger.error(
+            logger.info(
                 f"Validation error: {exception.message}{error_path_msg}",
                 extra={"validator": "body"},
             )