Bug #62669 for IO-Lambda: IO::Lambda::HTTP does not seem to be able to handle POST requests

Wed Nov 03 09:23:30 2010 MITHALDU [...] cpan.org - Ticket created

Subject:

IO::Lambda::HTTP does not seem to be able to handle POST requests

When trying to use IO::Lambda::HTTP to POST requests at urls, i only get error messages back. Lighttpd claims "400 malformed request", while google.com claims "411 content-length required. Looking at things with wireshark gave this: LWP::Useragent : ..)..m..7..=..E....y@........jJ}Mc@..P|x.@....P...c...POST / HTTP/1.1..TE: deflate,gzip;q=0.3..Connection: TE, close..Host: google.com..User-Agent: AssetManager/1.0..Content-Length: 5..Content- Type: application/x-www-form-urlencoded....id=34 IO::Lambda::HTTP : ..)..m..7..=..E.....@........jJ}Mc@..P.b./.5q.P...c...POST http:// google.com..User-Agent: AssetManager/1.0..Content-Length: 5..Content- Type: application/x-www-form-urlencoded....id=34 From that it seems that it simply omits the entire HTTP type header up until the Host: directive.

Wed Nov 03 10:44:45 2010 KARASIK [...] cpan.org - Correspondence added

Seems that the POST request was done using HTTP 0.9, is that intentional? Or indeed the module ignores protocol('HTTP/1.1') and other headers? Because if I take f.ex. eg/http-redirect.pl, and add $r-> headers-> header( 'Content-Length' => 0), there (yes, IO-Lambda doesn't fill these for you), google responds with error 405, not 411 , says POST inappropriate for URL, but Host: string is actually present in the http dump. Can you give a small script reproducing the problem?

Wed Nov 03 10:44:46 2010 The RT System itself - Status changed from 'new' to 'open'

Wed Nov 03 11:36:48 2010 MITHALDU [...] cpan.org - Correspondence added

Sorry, i forgot to add this. Here's the script i was running: https:// gist.github.com/661230

Wed Nov 03 11:52:24 2010 KARASIK [...] cpan.org - Correspondence added

Seems that this code for ( @chain ) { $_-> protocol('HTTP/1.1'); $_-> headers-> header('Host' => $_-> uri->host); } if added after line 24, does the thing. Is that a suitable solution? BTW, I've tried with HTTP 0.9 manually, eve-metrics.com doesn't support that anyway, even with GETs.

Wed Nov 03 11:59:28 2010 KARASIK [...] cpan.org - Correspondence added

ps - don't know if you've seen it already, but there exists a standard IO::Lambda::par() function for parallel execution with max lambdas

Thu Nov 04 12:17:47 2010 KARASIK [...] cpan.org - Correspondence added

assuming the patch solves the problem, I'm closing the ticket

Thu Nov 04 12:17:47 2010 KARASIK [...] cpan.org - Status changed from 'open' to 'resolved'