Bug #62712 for Net-Amazon-S3: Bug(?) with slashes in path being urlencoded as well

Thu Nov 04 23:50:33 2010 pavel [...] yepcorp.com - Ticket created

Subject:	Bug(?) with slashes in path being urlencoded as well
Date:	Thu, 4 Nov 2010 22:50:03 -0500
To:	bug-Net-Amazon-S3 [...] rt.cpan.org
From:	"Pavel A. Karoukin" <pavel [...] yepcorp.com>

Hello, Thank you for this module! I think I've found one bug. When storing object's key, key is being urlencoded. The problem is that sometimes I want to organize files on S3 in folders and Net::Amazon::S3::Request->_uri method urlencodes slashes in key. See patch attached for easy fix and let me know what do you think about it. Regards, Pavel

Message body is not shown because sender requested not to inline it.

Sat Nov 06 19:23:19 2010 pavel [...] yepcorp.com - Correspondence added

Subject:	Re: [rt.cpan.org #62712] AutoReply: Bug(?) with slashes in path being urlencoded as well
Date:	Sat, 6 Nov 2010 18:22:49 -0500
To:	bug-Net-Amazon-S3 [...] rt.cpan.org
From:	Pavel Karoukin <pavel [...] yepcorp.com>

I would be glad to become co-maintainer over this project too =) if interested - contact me at pavel@yepcorp.com

Sun Nov 07 19:51:51 2010 pavel [...] yepcorp.com - Correspondence added

Subject:	Re: [rt.cpan.org #62712] AutoReply: Bug(?) with slashes in path being urlencoded as well
Date:	Sun, 7 Nov 2010 18:51:20 -0600
To:	bug-Net-Amazon-S3 [...] rt.cpan.org
From:	Pavel Karoukin <pavel [...] yepcorp.com>

Another similar issue. When there is dot in key, it's being encoded as %2b. But, for example, Chrome decode this %2b BEFORE sending GET request and as a result requests fails, 'cos of wrong signature. Patch attached for excluding "." from being urlencoded. Regards, Pavel

Message body is not shown because sender requested not to inline it.

Thu Feb 24 04:16:17 2011 arthas [...] cpan.org - Correspondence added

Hello! Same issue with dots and MS Internet Explorer here. Thanks to Pavel Karoukin for the patch. Is there anyone willing take the maintenance of this module over and fix all the pending issues? Cheers.

Thu Feb 24 04:16:17 2011 The RT System itself - Status changed from 'new' to 'open'

Thu Feb 24 09:02:51 2011 pavel [...] yepcorp.com - Correspondence added

From:

pavel [...] yepcorp.com

I would be happy to maintain it, but have not clue how to upload new version of module so it appear in the same place current module resides. Any ideas? On Thu Feb 24 04:16:17 2011, ARTHAS wrote: Show quoted text

> Hello! > > Same issue with dots and MS Internet Explorer here. > > Thanks to Pavel Karoukin for the patch. > > Is there anyone willing take the maintenance of this module over and

fix Show quoted text

> all the pending issues? > > Cheers.

Wed Mar 02 05:28:42 2011 arthas [...] cpan.org - Correspondence added

Hello! Show quoted text

> I would be happy to maintain it, but have not clue how to upload new > version of module so it appear in the same place current module

resides. Show quoted text

> Any ideas?

It's probably as easy as to request authorization to upload in that namespace to the original author: Leon is usually easy to reach via email or IRC; if not, we can ask CPAN maintainers. To begin, since the project is on GitHub: https://github.com/acme/net-amazon-s3 you could begin forking it and applying the patches. I would be happy to help by joining the GitHub project you fork. Cheers, Michele.

Sun Jul 10 12:13:55 2011 me+pause [...] pedrofigueiredo.org - Correspondence added

Hi Pavel, thanks for your dot patch, I've included it in release 0.54. I didn't test your slashes fix, so it's not included, if I don't run into any issues with it, it'll be included in 0.55. Cheers, Pedro

Sun Jul 10 12:14:16 2011 me+pause [...] pedrofigueiredo.org - Taken

Thu Jul 14 14:30:38 2011 me+pause [...] pedrofigueiredo.org - Severity Critical added

Bug #62712 for Net-Amazon-S3: Bug(?) with slashes in path being urlencoded as well

Preferred bug tracker