Skip Menu |
 

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

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

People
Owner: Nobody in particular
Requestors: sbeck [...] gossamer-threads.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.63
Fixed in: (no value)

Attachments
http-status-parsing.patch



Subject: status line parsing
Download (untitled) / with headers
text/plain 266b
Hi, The current code to parse out the status line doesn't work probably. If in the first stream buffer coming in it can not match the status line, it clears the buffer and ends up putting everything in $res->content. Here is a patch that fixes this. Cheers, Scott
--- HTTP.pm 2004-02-27 10:59:36.000000000 -0800 +++ /usr/lib/perl5/site_perl/5.8.3/POE/Component/Client/HTTP.pm 2004-03-26 14:53:15.000000000 -0800 @@ -828,6 +828,10 @@ $request->[REQ_RESPONSE] = HTTP::Response->new(); $request->[REQ_STATE] = RS_IN_CONTENT; } + else { + # We need more data to match the status line + return; + } } # Parse the input for headers. This isn't in an else clause because
Download (untitled) / with headers
text/plain 125b
Thanks for the patch to handle interrupted status lines. It's applied and will appear in the next release. -- Rocco Caputo


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.