You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by "Andrew Gaul (Jira)" <ji...@apache.org> on 2024/11/14 01:55:00 UTC

[jira] [Commented] (JCLOUDS-1640) RequestAuthorizeSignatureV2 throws on folder with % in name

    [ https://issues.apache.org/jira/browse/JCLOUDS-1640?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17898097#comment-17898097 ] 

Andrew Gaul commented on JCLOUDS-1640:
--------------------------------------

This sounds plausible but without some kind of test I can't be sure.  Can you submit a PR with a test case that fails before but succeeds with your suggestion?

> RequestAuthorizeSignatureV2 throws on folder with % in name
> -----------------------------------------------------------
>
>                 Key: JCLOUDS-1640
>                 URL: https://issues.apache.org/jira/browse/JCLOUDS-1640
>             Project: jclouds
>          Issue Type: Bug
>    Affects Versions: 2.6.0
>            Reporter: Matthew Bellew
>            Assignee: Andrew Gaul
>            Priority: Major
>
> If I create a "folder" (or blob with a '%' in the folder part of its name).   S3BlobStore.list() will throw IllegalArgumentException( "URLDecoder: Incomplete trailing escape (%) pattern")
>  
> The problem seems to be with this line of code in RequestAutorizeSignatureV2.java:
> Multimap<String, String> params = queryParser().apply(request.getEndpoint().getQuery());
>  
> queryParser().apply() expects and encoded query string and will decode the string itself.  I believe getQuery() should be replaced with getRawQuery().



--
This message was sent by Atlassian Jira
(v8.20.10#820010)