Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 32260
Status: resolved
Priority: 0/
Queue: Net-Amazon-S3

People
Owner: Nobody in particular
Requestors: hanenkamp [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.41
Fixed in: (no value)



Subject: Leaving out critical information from response headers
I was trying to use Net::Amazon::S3, but I need to know when the key was last modified on the server. The module will allow the Content-Length, Content-Type, and ETag header through and any header starting with X-Amz-Meta-. However, this misses at least two potentially significant headers, Date, and Last-Modified. At least these two headers should be allowed to pass back out when getting a key. It is my opinion that it is a mistake to filter these at all because other headers might be significant to someone using the library or other important headers could be added. Throwing away information doesn't seem to be the right behavior to me.
The easy fix for this is to delete line 222 in 0.41. I have attached a patch.
--- old/Net/Amazon/S3/Bucket.pm 2008-01-11 16:10:37.000000000 -0600 +++ new/Net/Amazon/S3/Bucket.pm 2008-01-11 16:11:47.000000000 -0600 @@ -219,7 +219,6 @@ }; foreach my $header ( $response->headers->header_field_names ) { - next unless $header =~ /x-amz-meta-/i; $return->{ lc $header } = $response->header($header); }
Thanks, applied. This will be in the 0.45 release.