Merge pull request #213 from lsst-dm/tickets/DM-47333

DM-47333: Logger.exception() missing 1 required positional argument
lsst-dm · Nov 1, 2024 · cda43e4 · cda43e4
2 parents 94cc969 + 3bd2fea
commit cda43e4
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/python/activator/activator.py b/python/activator/activator.py
@@ -347,9 +347,9 @@ def next_visit_handler() -> tuple[str, int]:
     try:
         try:
             expected_visit = parse_next_visit(flask.request)
-        except ValueError as msg:
-            _log.exception()
-            return f"Bad Request: {msg}", 400
+        except ValueError as e:
+            _log.exception("Bad Request: %s", e)
+            return f"Bad Request: {e}", 400
         process_visit(expected_visit)
         return "Pipeline executed", 200
     except GracefulShutdownInterrupt:
@@ -524,7 +524,7 @@ def process_visit(expected_visit: FannedOutVisit):
 
 
 def invalid_visit(e: InvalidVisitError) -> tuple[str, int]:
-    _log.exception()
+    _log.exception("Invalid visit: %s", e)
     return f"Cannot process visit: {e}.", 422