Changeset 36156 in osm for applications/editors/josm/plugins

Timestamp:

2023-10-04T15:57:40+02:00 (12 months ago)

Author:

taylor.smock

Message:

Fix an off-by-one error in HTTP Range requests

File:

• applications/editors/josm/plugins/pmtiles/src/main/java/org/openstreetmap/josm/plugins/pmtiles/lib/PMTiles.java (modified) (3 diffs)

applications/editors/josm/plugins/pmtiles/src/main/java/org/openstreetmap/josm/plugins/pmtiles/lib/PMTiles.java

@@ -14 +14 @@
 import java.util.zip.InflaterInputStream;
 
-import jakarta.json.Json;
-import jakarta.json.JsonObject;
 import org.apache.commons.compress.compressors.brotli.BrotliCompressorInputStream;
 import org.apache.commons.compress.compressors.gzip.GzipCompressorInputStream;
@@ -21 +19 @@
 import org.openstreetmap.josm.plugins.pmtiles.lib.internal.HeaderParser;
 import org.openstreetmap.josm.tools.Utils;
+
+import jakarta.json.Json;
+import jakarta.json.JsonObject;
 
 /**
@@ -273 +274 @@
             }
         }
-        var request = HttpRequest.newBuilder(location).header("Range", "bytes=" + start + "-" + (start + length))
+        var request = HttpRequest.newBuilder(location).header("Range", "bytes=" + start + "-" + (start + length - 1))
                 .header("User-Agent", "JOSM PMTiles v1").GET().build();
         try {