From c2d54e7d59e3660dce2b1acc833b0b049ca4e9fb Mon Sep 17 00:00:00 2001 From: Ryan Malloy Date: Wed, 18 Feb 2026 18:15:41 -0700 Subject: [PATCH] Fix redundant except clause: ValueError is subclass of Exception except (ValueError, Exception) was redundant since ValueError already inherits from Exception. Simplified all 3 occurrences in the bbox resolution error handlers. --- src/mcgibs/server.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mcgibs/server.py b/src/mcgibs/server.py index cfb5b65..d340707 100644 --- a/src/mcgibs/server.py +++ b/src/mcgibs/server.py @@ -307,7 +307,7 @@ async def get_imagery( try: resolved_bbox = await _resolve_bbox(client, bbox, place) - except (ValueError, Exception) as exc: + except Exception as exc: return [{"type": "text", "text": str(exc)}] image_format = f"image/{format}" @@ -368,7 +368,7 @@ async def compare_dates( try: resolved_bbox = await _resolve_bbox(client, bbox, place) - except (ValueError, Exception) as exc: + except Exception as exc: return [{"type": "text", "text": str(exc)}] composite_bytes = await client.compare_dates( @@ -420,7 +420,7 @@ async def get_imagery_composite( try: resolved_bbox = await _resolve_bbox(client, bbox, place) - except (ValueError, Exception) as exc: + except Exception as exc: return [{"type": "text", "text": str(exc)}] image_bytes = await client.get_wms_composite(