Skip Menu |
 

This queue is for tickets about the POE-Component-Client-HTTP CPAN distribution.

Report information
The Basics
Id: 4521
Status: resolved
Priority: 0/
Queue: POE-Component-Client-HTTP

People
Owner: Nobody in particular
Requestors: poe [...] richardharman.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.59
Fixed in: (no value)



Subject: Component::Client::HTTP doesn't like ICY headers
Download (untitled) / with headers
text/plain 402b
The status line parsing code expects "HTTP" to be there, and if it isn't, any http headers following it are ignored. Changing: if ( $request->[REQ_BUFFER] =~ s/^(HTTP\/[0-9\.]+)?\s*(\d+)\s*(.*?)([\x0D\x0A]+)([^\x0D\x0A])/$5/ ) { To: if ( $request->[REQ_BUFFER] =~ s/^([A-Z]+(?:\/[0-9\.]+)?)?\s*(\d+)\s*(.*?)([\x0D\x0A]+)([^\x0D\x0A])/$5/x ) { fixes it.
Download (untitled) / with headers
text/plain 185b
Thanks for the patch, and thanks for forwarding it through the ticket system. It's applied and will be in the next release. -- Rocco Caputo - rcaputo@pobox.com - http://poe.perl.org/


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.