Use file extension from content-type instead

openaddresses · Sep 9, 2023 · 263afce · 263afce
1 parent b6b6f19
commit 263afce
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/openaddr/cache.py b/openaddr/cache.py
@@ -207,8 +207,11 @@ def guess_url_file_extension(url):
                     if path_ext == attachment_ext:
                         _L.debug('Content-Disposition agrees: "{}"'.format(match.group('filename')))
                     else:
-                        _L.debug('Content-Disposition disagrees: "{}", using "{}" instead'.format(match.group('filename'), attachment_ext))
-                        path_ext = attachment_ext
+                        _L.debug('Content-Disposition disagrees: "{}" says we should use "{}", using "{}" instead'.format(
+                            match.group('filename'),
+                            attachment_ext,
+                            path_ext,
+                        ))
 
         if not path_ext:
             #