Issues

ZF-10570: Zend_Service_Amazon_S3 serial request bug with Content-Encoding header set

Issue Type: Bug Created: 2010-10-17T09:44:47.000+0000 Last Updated: 2010-10-19T10:37:25.000+0000 Status: Resolved Fix version(s): - 1.11.0 (02/Nov/10)

Reporter: Nick Miller (nicholas.j.miller) Assignee: Matthew Weier O'Phinney (matthew) Tags: - Zend_Service_Amazon

Related issues: Attachments:

Description

Zend_Service_Amazon_S3::_makeRequest() does not reset the 'Content-Encoding' header. This creates a problem when content with gzip content-encoding is uploaded to S3, then subsequent, non compressed, data is uploaded. I have included a patch below to address the issue.

==== Dif File ===

This patch file was generated by NetBeans IDE

It uses platform neutral UTF-8 encoding and \n newlines.

--- Base (BASE) +++ Locally Modified (Based On LOCAL) @@ -559,7 +559,8 @@ $client->setHeaders(array('Content-MD5' => null, 'Expect' => null, 'Range' => null, - 'x-amz-acl' => null)); + 'x-amz-acl' => null, + 'Content-Encoding' => null));

     $client->setHeaders($headers);

Comments

Posted by Matthew Weier O'Phinney (matthew) on 2010-10-19T10:33:04.000+0000

Patch applied. In the future, please create your diff file from the top-level ZF directory (the one that contains library, tests, documentation, etc), and/or specify the file that needs to be patched. Also, please include unit tests in the future.

Posted by Nick Miller (nicholas.j.miller) on 2010-10-19T10:37:25.000+0000

Hi Matthew,

I'll make sure to do that for any subsequent bugs.

Thanks, Nick

Have you found an issue?

See the Overview section for more details.

Copyright

© 2006-2016 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.

Contacts