Skip Menu |

This queue is for tickets about the Net-Amazon-Signature-V4 CPAN distribution.

Report information
The Basics
Id: 128728
Status: resolved
Priority: 0/
Queue: Net-Amazon-Signature-V4
People
Owner: Nobody in particular
Requestors: dev [...] timmullin.com
Cc: tim [...] cpanel.net
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.19
Fixed in: 0.20

History Show all quoted text
  Wed Mar 06 12:32:02 2019 dev [...] timmullin.com - Ticket created
CC: tim [...] cpanel.net
Subject: _simplify_uri performing uri_encode breaks S3
The fix added for bug #126025 breaks connecting to Amazon S3 when the URI contains special characters have been encoded. Apparently, every service but S3, needs to have those characters double-encoded. From Step 2 in "creating a canonical request" in the following document: https://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical-request.html "Normalize URI paths according to RFC 3986. Remove redundant and relative path components. Each path segment must be URI-encoded twice (except for Amazon S3 which only gets URI-encoded once)." I have a proposed fix for this and will be submitting a pull request shortly.
  Wed Mar 06 12:45:49 2019 dev [...] timmullin.com - Correspondence added
RT-Send-CC: tim [...] cpanel.net
On Wed Mar 06 12:32:02 2019, dev@timmullin.com wrote: Show quoted text
> The fix added for bug #126025 breaks connecting to Amazon S3 when the > URI contains special characters have been encoded. Apparently, every > service but S3, needs to have those characters double-encoded. > > From Step 2 in "creating a canonical request" in the following > document: > https://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical- > request.html > > "Normalize URI paths according to RFC 3986. Remove redundant and > relative path components. Each path segment must be URI-encoded twice > (except for Amazon S3 which only gets URI-encoded once)." > > I have a proposed fix for this and will be submitting a pull request > shortly.
I have submitting the following pull request: https://github.com/Grinnz/Net-Amazon-Signature-V4/pull/7
  Fri Feb 21 12:44:29 2020 DBOOK [...] cpan.org - Correspondence added
This has been addressed in 0.20.
  Fri Feb 21 12:44:30 2020 The RT System itself - Status changed from 'new' to 'open'
  Fri Feb 21 12:44:30 2020 DBOOK [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri Feb 21 12:44:31 2020 DBOOK [...] cpan.org - Fixed in 0.20 added