Skip checksum validation for multifile downloads where no checksum is provided

kylstevenson · kylstevenson · commit c53a2d399096 · 2026-01-13T16:36:55.000Z
diff --git a/src/main/java/io/github/jpmorganchase/fusion/api/request/PartFetcher.java b/src/main/java/io/github/jpmorganchase/fusion/api/request/PartFetcher.java
@@ -28,7 +28,6 @@
 public class PartFetcher {
 
     private static final String HEAD_PATH = "%s/operationType/download";
-    private static final String HEAD_PATH_FOR_PART = "%s?downloadPartNumber=%d";
 
     Client client;
     FusionTokenProvider credentials;
@@ -123,7 +122,9 @@ private String getSinglePartPath(PartRequest pr) {
     }
 
     private String getMultiPartPath(PartRequest pr) {
-        return String.format(HEAD_PATH_FOR_PART, getHeadPath(pr), pr.getPartNo());
+        String headPath = getHeadPath(pr);
+        String separator = headPath.contains("?") ? "&" : "?";
+        return headPath + separator + "downloadPartNumber=" + pr.getPartNo();
     }
 
     private String getHeadPath(PartRequest pr) {

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,6 @@`
`28`	`28`	`public class PartFetcher {`
`29`	`29`
`30`	`30`	`private static final String HEAD_PATH = "%s/operationType/download";`
`31`		`- private static final String HEAD_PATH_FOR_PART = "%s?downloadPartNumber=%d";`
`32`	`31`
`33`	`32`	`Client client;`
`34`	`33`	`FusionTokenProvider credentials;`
`@@ -123,7 +122,9 @@ private String getSinglePartPath(PartRequest pr) {`
`123`	`122`	`}`
`124`	`123`
`125`	`124`	`private String getMultiPartPath(PartRequest pr) {`
`126`		`- return String.format(HEAD_PATH_FOR_PART, getHeadPath(pr), pr.getPartNo());`
	`125`	`+ String headPath = getHeadPath(pr);`
	`126`	`+ String separator = headPath.contains("?") ? "&" : "?";`
	`127`	`+ return headPath + separator + "downloadPartNumber=" + pr.getPartNo();`
`127`	`128`	`}`
`128`	`129`
`129`	`130`	`private String getHeadPath(PartRequest pr) {`