Fix bug related to error type handling

wazuh · May 23, 2024 · 548f612 · 548f612
1 parent 023f6ce
commit 548f612
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/api/api/error_handler.py b/api/api/error_handler.py
@@ -155,10 +155,9 @@ async def problem_error_handler(request: ConnexionRequest, exc: exceptions.Probl
     """
     problem = {
         "title": exc.title if exc.title else 'Bad Request',
-        "detail": exc.detail if isinstance(exc.detail, dict) \
-                    else _cleanup_detail_field(exc.detail)
+        "detail": exc.detail if isinstance(exc.detail, dict) else _cleanup_detail_field(exc.detail)
     }
-    problem.update({"type": exc.type} if exc.type else {})
+    problem.update({"type": exc.type} if (exc.type and exc.type != 'about:blank') else {})
     problem.update(exc.ext if exc.ext else {})
     if isinstance(problem['detail'], dict):
         for field in ['status', 'type']:

diff --git a/api/api/test/test_error_handler.py b/api/api/test/test_error_handler.py
@@ -163,7 +163,7 @@ async def test_problem_error_handler(title, detail, ext, error_type, mock_reques
         detail = _cleanup_detail_field(detail)
     problem = {}
     problem.update({'title': title} if title else {'title': 'Bad Request'})
-    problem.update({'type': error_type} if error_type else {})
+    problem.update({'type': error_type} if (error_type and error_type != 'about:blank') else {})
     problem.update({'detail': detail} if detail else {})
     problem.update(ext if ext else {})
     problem.update({'error': problem.pop('code')} if 'code' in problem else {})