Fix proxy support header parsing problem #49

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

As referenced here:

https://forums.aws.amazon.com/thread.jspa?messageID=227413

This patch fixes a problem parsing the proxy header. Otherwise, the tail of the header ends up in the $body and it throws parse errors such as:

Warning: SimpleXMLElement::__construct(): Entity: line 1: parser error : Start tag expected, '<' not found in /usr/share/pear/AWSSDKforPHP/sdk.class.php on line 1275

Warning: SimpleXMLElement::__construct(): 012 22:53:43 GMT in /usr/share/pear/AWSSDKforPHP/sdk.class.php on line 1275

Warning: SimpleXMLElement::__construct(): ^ in /usr/share/pear/AWSSDKforPHP/sdk.class.php on line 1275

Fatal error: Uncaught exception 'Parser_Exception' with message 'String could not be parsed as XML' in /usr/share/pear/AWSSDKforPHP/sdk.class.php:1279
Stack trace:
#0 /usr/share/pear/AWSSDKforPHP/sdk.class.php(1048): CFRuntime->parse_callback('012 22:53:43 GM...')
#1 /usr/share/pear/AWSSDKforPHP/services/ec2.class.php(3425): CFRuntime->authenticate('DescribeVolumes', Array)

jeskew commented Mar 2, 2016

Closing, as we're only iterating on the versions 2 & 3 of the AWS SDK for PHP.

Please feel free to maintain a fork with whatever modifications you would like to make.

@jeskew jeskew closed this Mar 2, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment