Skip to content

Commit d296fe9

Browse files
chore:fix sign bug
1 parent 222972a commit d296fe9

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

volcengine-java-sdk-core/src/main/java/com/volcengine/sign/VolcstackSign.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -377,20 +377,16 @@ public Map<String, String> presign(Map<String, String> queryParams, String host)
377377
presignedParams.put("X-Security-Token", credentials.getSessionToken());
378378
}
379379

380-
// Re-collect all parameter keys (including X-Security-Token) to build canonical request
381-
List<String> allKeys = new ArrayList<>(presignedParams.keySet());
382-
Collections.sort(allKeys);
383-
384380
// Build canonical request
385381
StringBuilder canonicalRequest = new StringBuilder();
386382
canonicalRequest.append(this.method);
387383
canonicalRequest.append("\n");
388384
canonicalRequest.append("/");
389385
canonicalRequest.append("\n");
390386

391-
// Canonical Query String - concatenate all parameters after sorting
387+
// Canonical Query String - only include keys listed in X-SignedQueries
392388
StringBuilder canonicalQueryString = new StringBuilder();
393-
for (String key : allKeys) {
389+
for (String key : queryKeys) {
394390
canonicalQueryString.append(signStringEncoder(key));
395391
canonicalQueryString.append("=");
396392
canonicalQueryString.append(signStringEncoder(presignedParams.get(key)));

0 commit comments

Comments
 (0)