(jclouds) branch master updated: JCLOUDS-1640: Use raw query to sign S3 V2 request

[gaul]

This is an automated email from the ASF dual-hosted git repository.

gaul pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jclouds.git


The following commit(s) were added to refs/heads/master by this push:
     new 07a4ed72cc JCLOUDS-1640: Use raw query to sign S3 V2 request
07a4ed72cc is described below

commit 07a4ed72ccb0a75835a08ad414278d2c7ca40508
Author: Andrew Gaul <and...@gaul.org>
AuthorDate: Sun Dec 22 17:30:34 2024 -0500

    JCLOUDS-1640: Use raw query to sign S3 V2 request
    
    This prevents errors when listing objects with % in the key names.
    Found via S3Proxy AwsSdkTest.testSpecialCharacters.
---
 .../main/java/org/jclouds/s3/filters/RequestAuthorizeSignatureV2.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/apis/s3/src/main/java/org/jclouds/s3/filters/RequestAuthorizeSignatureV2.java 
b/apis/s3/src/main/java/org/jclouds/s3/filters/RequestAuthorizeSignatureV2.java
index d61db3ffd1..8202ab2359 100644
--- 
a/apis/s3/src/main/java/org/jclouds/s3/filters/RequestAuthorizeSignatureV2.java
+++ 
b/apis/s3/src/main/java/org/jclouds/s3/filters/RequestAuthorizeSignatureV2.java
@@ -253,7 +253,7 @@ public class RequestAuthorizeSignatureV2 implements 
RequestAuthorizeSignature, R
       // ...however, there are a few exceptions that must be included in the
       // signed URI.
       if (request.getEndpoint().getQuery() != null) {
-         Multimap<String, String> params = 
queryParser().apply(request.getEndpoint().getQuery());
+         Multimap<String, String> params = 
queryParser().apply(request.getEndpoint().getRawQuery());
          char separator = '?';
          for (String paramName : 
Ordering.natural().sortedCopy(params.keySet())) {
             // Skip any parameters that aren't part of the canonical signed 
string