Changeset 9174 in josm for trunk/src/org/openstreetmap

Timestamp:

2015-12-27T00:07:00+01:00 (9 years ago)

Author:

simon04

Message:

see #12231 - Update unit tests, fix regression

Location:

trunk/src/org/openstreetmap/josm

Files:

• io/OsmServerReader.java (modified) (1 diff)
• tools/HttpClient.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/io/OsmServerReader.java

@@ -152 +152 @@
                 }
 
+                activeConnection.uncompressAccordingToContentDisposition(uncompressAccordingToContentDisposition);
                 InputStream in = new ProgressInputStream(activeConnection, progressMonitor);
-                if (uncompressAccordingToContentDisposition) {
-                    activeConnection.uncompressAccordingToContentDisposition(true);
-                }
                 return in;
             } catch (OsmTransferException e) {

trunk/src/org/openstreetmap/josm/tools/HttpClient.java

@@ -183 +183 @@
             }
             in = "gzip".equalsIgnoreCase(getContentEncoding()) ? new GZIPInputStream(in) : in;
+            Compression compression = Compression.NONE;
             if (uncompress) {
                 final String contentType = getContentType();
                 Main.debug("Uncompressing input stream according to Content-Type header: {0}", contentType);
-                in = Compression.forContentType(contentType).getUncompressedInputStream(in);
-            }
-            if (uncompressAccordingToContentDisposition) {
+                compression = Compression.forContentType(contentType);
+            }
+            if (uncompressAccordingToContentDisposition && Compression.NONE.equals(compression)) {
                 final String contentDisposition = getHeaderField("Content-Disposition");
                 final Matcher matcher = Pattern.compile("filename=\"([^\"]+)\"").matcher(contentDisposition);
                 if (matcher.find()) {
                     Main.debug("Uncompressing input stream according to Content-Disposition header: {0}", contentDisposition);
-                    in = Compression.byExtension(matcher.group(1)).getUncompressedInputStream(in);
+                    compression = Compression.byExtension(matcher.group(1));
                 }
             }
+            in = compression.getUncompressedInputStream(in);
             return in;
         }