Merge branch 'master' of github.com:openaddresses/batch-machine into …

…internal-geojson

CHANGELOG

@@ -4,15 +4,17 @@
 - Rewrite confrom to natively output GeoJSON+LD
 - Rewrite Preview module to expect GeoJSON+LD
 
-2023-09-06 v8.3.0
-- Update to next major alpine version
-- Remove mock library in favour of unittest.mock
-- Update all python deps to latest versions
+2023-09-09 v8.3.1
+- Use the file extension from the content-type header when it disagrees with content-disposition in https://github.com/openaddresses/batch-machine/pull/54
+
+2023-09-05 v8.3.0
+- Update Mapbox Vector Tile dependency in https://github.com/openaddresses/batch-machine/pull/52
+- Make GeoJSON+LD output a default in https://github.com/openaddresses/batch-machine/pull/51
+- Update TippeCanoe to latest version in https://github.com/openaddresses/batch-machine/pull/50
 
 2023-09-05 v8.2.4
 - Update TippeCanoe to latest version
 - Make GeoJSON+LD output a default
-
 2023-09-02 v8.2.3
 - Fix properties in geojson-ld features in https://github.com/openaddresses/batch-machine/pull/49
 - Remove extra whitespace in geojson-ld output in https://github.com/openaddresses/batch-machine/pull/48

openaddr/cache.py

@@ -208,8 +208,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: "{}"'.format(match.group('filename')))
-                        path_ext = False
+                        _L.debug('Content-Disposition disagrees: "{}" says we should use "{}", using "{}" instead'.format(
+                            match.group('filename'),
+                            attachment_ext,
+                            path_ext,
+                        ))
 
         if not path_ext:
             #