Switch False gzip Error to Warning (#62)

* Switch False gzip Error to Warning By logging errors, events get sent to analytics captures (like Sentry) that aren't actually a problem since the usp can continue to parse the non-gzipped data * Update usp/helpers.py * Add changelog entry --------- Co-authored-by: Freddy Heppell <[email protected]>
GateNLP · Jan 23, 2025 · a9d9169 · a9d9169
1 parent ae51209
commit a9d9169
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -1,6 +1,13 @@
 Changelog
 =========
 
+Upcoming
+--------
+
+**Bug Fixes**
+
+- Changed log level when a suspected gzipped sitemap can't be un-gzipped from `error` to `warning`, since parsing can usually continue (:pr:`62` by :user:`redreceipt`)
+
 v1.1.0 (2025-01-20)
 -------------------
 

diff --git a/usp/helpers.py b/usp/helpers.py
@@ -247,7 +247,7 @@ def ungzipped_response_content(
             data = gunzip(data)
         except GunzipException as ex:
             # In case of an error, just assume that it's one of the non-gzipped sitemaps with ".gz" extension
-            log.error(
+            log.warning(
                 f"Unable to gunzip response {response}, maybe it's a non-gzipped sitemap: {ex}"
             )