Do not assume 'content-encoding' header is set.

bluesky · Apr 24, 2024 · 62fd8da · 62fd8da
1 parent 0cd0407
commit 62fd8da
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/tiled/server/metrics.py b/tiled/server/metrics.py
@@ -125,13 +125,14 @@ def capture_request_metrics(request, response):
             metrics["read"]["pack"]
         )
     if "compress" in metrics:
-        encoding = response.headers["content-encoding"]
-        COMPRESSION_DURATION.labels(
-            method=method, code=code, endpoint=endpoint, encoding=encoding
-        ).observe(metrics["compress"]["dur"])
-        COMPRESSION_RATIO.labels(
-            method=method, code=code, endpoint=endpoint, encoding=encoding
-        ).observe(metrics["compress"]["ratio"])
+        encoding = response.headers.get("content-encoding")
+        if encoding:
+            COMPRESSION_DURATION.labels(
+                method=method, code=code, endpoint=endpoint, encoding=encoding
+            ).observe(metrics["compress"]["dur"])
+            COMPRESSION_RATIO.labels(
+                method=method, code=code, endpoint=endpoint, encoding=encoding
+            ).observe(metrics["compress"]["ratio"])
 
 
 @lru_cache()