ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/AnyEvent-HTTP/HTTP.pm
(Generate patch)

Comparing AnyEvent-HTTP/HTTP.pm (file contents):
Revision 1.25 by root, Mon Jul 21 05:42:07 2008 UTC vs.
Revision 1.37 by root, Fri Nov 21 08:20:14 2008 UTC

… …
48use AnyEvent::Socket (); 48use AnyEvent::Socket ();
49use AnyEvent::Handle (); 49use AnyEvent::Handle ();
50 50
51use base Exporter::; 51use base Exporter::;
52 52
53our $VERSION = '1.03'; 53our $VERSION = '1.11';
54 54
55our @EXPORT = qw(http_get http_post http_head http_request); 55our @EXPORT = qw(http_get http_post http_head http_request);
56 56
57our $USERAGENT = "Mozilla/5.0 (compatible; AnyEvent::HTTP/$VERSION; +http://software.schmorp.de/pkg/AnyEvent)"; 57our $USERAGENT = "Mozilla/5.0 (compatible; AnyEvent::HTTP/$VERSION; +http://software.schmorp.de/pkg/AnyEvent)";
58our $MAX_RECURSE = 10; 58our $MAX_RECURSE = 10;
… …
71my %CO_SLOT; # number of open connections, and wait queue, per host 71my %CO_SLOT; # number of open connections, and wait queue, per host
72 72
73=item http_get $url, key => value..., $cb->($data, $headers) 73=item http_get $url, key => value..., $cb->($data, $headers)
74 74
75Executes an HTTP-GET request. See the http_request function for details on 75Executes an HTTP-GET request. See the http_request function for details on
76additional parameters. 76additional parameters and the return value.
77 77
78=item http_head $url, key => value..., $cb->($data, $headers) 78=item http_head $url, key => value..., $cb->($data, $headers)
79 79
80Executes an HTTP-HEAD request. See the http_request function for details on 80Executes an HTTP-HEAD request. See the http_request function for details
81additional parameters. 81on additional parameters and the return value.
82 82
83=item http_post $url, $body, key => value..., $cb->($data, $headers) 83=item http_post $url, $body, key => value..., $cb->($data, $headers)
84 84
85Executes an HTTP-POST request with a request body of C<$bod>. See the 85Executes an HTTP-POST request with a request body of C<$body>. See the
86http_request function for details on additional parameters. 86http_request function for details on additional parameters and the return
87value.
87 88
88=item http_request $method => $url, key => value..., $cb->($data, $headers) 89=item http_request $method => $url, key => value..., $cb->($data, $headers)
89 90
90Executes a HTTP request of type C<$method> (e.g. C<GET>, C<POST>). The URL 91Executes a HTTP request of type C<$method> (e.g. C<GET>, C<POST>). The URL
91must be an absolute http or https URL. 92must be an absolute http or https URL.
93
94When called in void context, nothing is returned. In other contexts,
95C<http_request> returns a "cancellation guard" - you have to keep the
96object at least alive until the callback get called. If the object gets
97destroyed before the callbakc is called, the request will be cancelled.
92 98
93The callback will be called with the response data as first argument 99The callback will be called with the response data as first argument
94(or C<undef> if it wasn't available due to errors), and a hash-ref with 100(or C<undef> if it wasn't available due to errors), and a hash-ref with
95response headers as second argument. 101response headers as second argument.
96 102
… …
98headers, the "pseudo-headers" C<HTTPVersion>, C<Status> and C<Reason> 104headers, the "pseudo-headers" C<HTTPVersion>, C<Status> and C<Reason>
99contain the three parts of the HTTP Status-Line of the same name. The 105contain the three parts of the HTTP Status-Line of the same name. The
100pseudo-header C<URL> contains the original URL (which can differ from the 106pseudo-header C<URL> contains the original URL (which can differ from the
101requested URL when following redirects). 107requested URL when following redirects).
102 108
103If the server sends a header multiple lines, then their contents will be 109If the server sends a header multiple times, then their contents will be
104joined together with C<\x00>. 110joined together with a comma (C<,>), as per the HTTP spec.
105 111
106If an internal error occurs, such as not being able to resolve a hostname, 112If an internal error occurs, such as not being able to resolve a hostname,
107then C<$data> will be C<undef>, C<< $headers->{Status} >> will be C<599> 113then C<$data> will be C<undef>, C<< $headers->{Status} >> will be C<599>
108and the C<Reason> pseudo-header will contain an error message. 114and the C<Reason> pseudo-header will contain an error message.
109 115
… …
188 my ($body, $hdr) = @_; 194 my ($body, $hdr) = @_;
189 use Data::Dumper; 195 use Data::Dumper;
190 print Dumper $hdr; 196 print Dumper $hdr;
191 } 197 }
192 ; 198 ;
199
200Example: make another simple HTTP GET request, but immediately try to
201cancel it.
202
203 my $request = http_request GET => "http://www.nethype.de/", sub {
204 my ($body, $hdr) = @_;
205 print "$body\n";
206 };
207
208 undef $request;
193 209
194=cut 210=cut
195 211
196sub _slot_schedule; 212sub _slot_schedule;
197sub _slot_schedule($) { 213sub _slot_schedule($) {
… …
221 push @{ $CO_SLOT{$_[0]}[1] }, $_[1]; 237 push @{ $CO_SLOT{$_[0]}[1] }, $_[1];
222 238
223 _slot_schedule $_[0]; 239 _slot_schedule $_[0];
224} 240}
225 241
242our $qr_nl = qr<\015?\012>;
243our $qr_nlnl = qr<\015?\012\015?\012>;
244
226sub http_request($$@) { 245sub http_request($$@) {
227 my $cb = pop; 246 my $cb = pop;
228 my ($method, $url, %arg) = @_; 247 my ($method, $url, %arg) = @_;
229 248
230 my %hdr; 249 my %hdr;
… …
245 my $proxy = $arg{proxy} || $PROXY; 264 my $proxy = $arg{proxy} || $PROXY;
246 my $timeout = $arg{timeout} || $TIMEOUT; 265 my $timeout = $arg{timeout} || $TIMEOUT;
247 266
248 $hdr{"user-agent"} ||= $USERAGENT; 267 $hdr{"user-agent"} ||= $USERAGENT;
249 268
250 my ($scheme, $authority, $upath, $query, $fragment) = 269 my ($uscheme, $uauthority, $upath, $query, $fragment) =
251 $url =~ m|(?:([^:/?#]+):)?(?://([^/?#]*))?([^?#]*)(?:\?([^#]*))?(?:#(.*))?|; 270 $url =~ m|(?:([^:/?#]+):)?(?://([^/?#]*))?([^?#]*)(?:\?([^#]*))?(?:#(.*))?|;
252 271
253 $scheme = lc $scheme; 272 $uscheme = lc $uscheme;
254 273
255 my $uport = $scheme eq "http" ? 80 274 my $uport = $uscheme eq "http" ? 80
256 : $scheme eq "https" ? 443 275 : $uscheme eq "https" ? 443
257 : return $cb->(undef, { Status => 599, Reason => "only http and https URL schemes supported", URL => $url }); 276 : return $cb->(undef, { Status => 599, Reason => "only http and https URL schemes supported", URL => $url });
258 277
259 $hdr{referer} ||= "$scheme://$authority$upath"; # leave out fragment and query string, just a heuristic 278 $hdr{referer} ||= "$uscheme://$uauthority$upath"; # leave out fragment and query string, just a heuristic
260 279
261 $authority =~ /^(?: .*\@ )? ([^\@:]+) (?: : (\d+) )?$/x 280 $uauthority =~ /^(?: .*\@ )? ([^\@:]+) (?: : (\d+) )?$/x
262 or return $cb->(undef, { Status => 599, Reason => "unparsable URL", URL => $url }); 281 or return $cb->(undef, { Status => 599, Reason => "unparsable URL", URL => $url });
263 282
264 my $uhost = $1; 283 my $uhost = $1;
265 $uport = $2 if defined $2; 284 $uport = $2 if defined $2;
266 285
… …
269 288
270 $upath =~ s%^/?%/%; 289 $upath =~ s%^/?%/%;
271 290
272 # cookie processing 291 # cookie processing
273 if (my $jar = $arg{cookie_jar}) { 292 if (my $jar = $arg{cookie_jar}) {
274 %$jar = () if $jar->{version} < 1; 293 %$jar = () if $jar->{version} != 1;
275 294
276 my @cookie; 295 my @cookie;
277 296
278 while (my ($chost, $v) = each %$jar) { 297 while (my ($chost, $v) = each %$jar) {
298 if ($chost =~ /^\./) {
279 next unless $chost eq substr $uhost, -length $chost; 299 next unless $chost eq substr $uhost, -length $chost;
280 next unless $chost =~ /^\./; 300 } elsif ($chost =~ /\./) {
301 next unless $chost eq $uhost;
302 } else {
303 next;
304 }
281 305
282 while (my ($cpath, $v) = each %$v) { 306 while (my ($cpath, $v) = each %$v) {
283 next unless $cpath eq substr $upath, 0, length $cpath; 307 next unless $cpath eq substr $upath, 0, length $cpath;
284 308
285 while (my ($k, $v) = each %$v) { 309 while (my ($k, $v) = each %$v) {
286 next if $scheme ne "https" && exists $v->{secure}; 310 next if $uscheme ne "https" && exists $v->{secure};
311 my $value = $v->{value};
312 $value =~ s/([\\"])/\\$1/g;
287 push @cookie, "$k=$v->{value}"; 313 push @cookie, "$k=\"$value\"";
288 } 314 }
289 } 315 }
290 } 316 }
291 317
292 $hdr{cookie} = join "; ", @cookie 318 $hdr{cookie} = join "; ", @cookie
293 if @cookie; 319 if @cookie;
294 } 320 }
295 321
296 my ($rhost, $rport, $rpath); # request host, port, path 322 my ($rhost, $rport, $rscheme, $rpath); # request host, port, path
297 323
298 if ($proxy) { 324 if ($proxy) {
299 ($rhost, $rport, $scheme) = @$proxy; 325 ($rhost, $rport, $rscheme, $rpath) = (@$proxy, $url);
300 $rpath = $url; 326
327 # don't support https requests over https-proxy transport,
328 # can't be done with tls as spec'ed.
329 $rscheme = "http" if $uscheme eq "https" && $rscheme eq "https";
301 } else { 330 } else {
302 ($rhost, $rport, $rpath) = ($uhost, $uport, $upath); 331 ($rhost, $rport, $rscheme, $rpath) = ($uhost, $uport, $uscheme, $upath);
303 $hdr{host} = $uhost;
304 } 332 }
305 333
334 $hdr{host} = $uhost;
306 $hdr{"content-length"} = length $arg{body}; 335 $hdr{"content-length"} = length $arg{body};
307 336
308 my %state = (connect_guard => 1); 337 my %state = (connect_guard => 1);
309 338
310 _get_slot $uhost, sub { 339 _get_slot $uhost, sub {
… …
313 return unless $state{connect_guard}; 342 return unless $state{connect_guard};
314 343
315 $state{connect_guard} = AnyEvent::Socket::tcp_connect $rhost, $rport, sub { 344 $state{connect_guard} = AnyEvent::Socket::tcp_connect $rhost, $rport, sub {
316 $state{fh} = shift 345 $state{fh} = shift
317 or return $cb->(undef, { Status => 599, Reason => "$!", URL => $url }); 346 or return $cb->(undef, { Status => 599, Reason => "$!", URL => $url });
347 pop; # free memory, save a tree
318 348
319 delete $state{connect_guard}; # reduce memory usage, save a tree 349 return unless delete $state{connect_guard};
320 350
321 # get handle 351 # get handle
322 $state{handle} = new AnyEvent::Handle 352 $state{handle} = new AnyEvent::Handle
323 fh => $state{fh}, 353 fh => $state{fh},
324 ($scheme eq "https" ? (tls => "connect") : ()); 354 timeout => $timeout;
325 355
326 # limit the number of persistent connections 356 # limit the number of persistent connections
357 # keepalive not yet supported
327 if ($KA_COUNT{$_[1]} < $MAX_PERSISTENT_PER_HOST) { 358 if ($KA_COUNT{$_[1]} < $MAX_PERSISTENT_PER_HOST) {
328 ++$KA_COUNT{$_[1]}; 359 ++$KA_COUNT{$_[1]};
329 $state{handle}{ka_count_guard} = AnyEvent::Util::guard { --$KA_COUNT{$_[1]} }; 360 $state{handle}{ka_count_guard} = AnyEvent::Util::guard {
361 --$KA_COUNT{$_[1]}
362 };
330 $hdr{connection} = "keep-alive"; 363 $hdr{connection} = "keep-alive";
331 delete $hdr{connection}; # keep-alive not yet supported
332 } else { 364 } else {
333 delete $hdr{connection}; 365 delete $hdr{connection};
334 } 366 }
335 367
336 # (re-)configure handle 368 # (re-)configure handle
337 $state{handle}->timeout ($timeout);
338 $state{handle}->on_error (sub { 369 $state{handle}->on_error (sub {
339 my $errno = "$!"; 370 my $errno = "$!";
340 %state = (); 371 %state = ();
341 $cb->(undef, { Status => 599, Reason => $errno, URL => $url }); 372 $cb->(undef, { Status => 599, Reason => $errno, URL => $url });
342 }); 373 });
343 $state{handle}->on_eof (sub { 374 $state{handle}->on_eof (sub {
344 %state = (); 375 %state = ();
345 $cb->(undef, { Status => 599, Reason => "unexpected end-of-file", URL => $url }); 376 $cb->(undef, { Status => 599, Reason => "unexpected end-of-file", URL => $url });
346 }); 377 });
347 378
379 $state{handle}->starttls ("connect") if $rscheme eq "https";
380
381 # handle actual, non-tunneled, request
382 my $handle_actual_request = sub {
383 $state{handle}->starttls ("connect") if $uscheme eq "https" && !exists $state{handle}{tls};
384
348 # send request 385 # send request
349 $state{handle}->push_write ( 386 $state{handle}->push_write (
350 "$method $rpath HTTP/1.0\015\012" 387 "$method $rpath HTTP/1.0\015\012"
351 . (join "", map "$_: $hdr{$_}\015\012", keys %hdr) 388 . (join "", map "\u$_: $hdr{$_}\015\012", keys %hdr)
352 . "\015\012" 389 . "\015\012"
353 . (delete $arg{body}) 390 . (delete $arg{body})
354 );
355
356 %hdr = (); # reduce memory usage, save a kitten
357
358 # status line
359 $state{handle}->push_read (line => qr/\015?\012/, sub {
360 $_[1] =~ /^HTTP\/([0-9\.]+) \s+ ([0-9]{3}) (?: \s+ ([^\015\012]*) )?/ix
361 or return (%state = (), $cb->(undef, { Status => 599, Reason => "invalid server response ($_[1])", URL => $url }));
362
363 my %hdr = ( # response headers
364 HTTPVersion => "\x00$1",
365 Status => "\x00$2",
366 Reason => "\x00$3",
367 URL => "\x00$url"
368 ); 391 );
369 392
393 %hdr = (); # reduce memory usage, save a kitten
394
395 # status line
396 $state{handle}->push_read (line => $qr_nl, sub {
397 $_[1] =~ /^HTTP\/([0-9\.]+) \s+ ([0-9]{3}) (?: \s+ ([^\015\012]*) )?/ix
398 or return (%state = (), $cb->(undef, { Status => 599, Reason => "invalid server response ($_[1])", URL => $url }));
399
400 my %hdr = ( # response headers
401 HTTPVersion => ",$1",
402 Status => ",$2",
403 Reason => ",$3",
404 URL => ",$url"
405 );
406
370 # headers, could be optimized a bit 407 # headers, could be optimized a bit
371 $state{handle}->unshift_read (line => qr/\015?\012\015?\012/, sub { 408 $state{handle}->unshift_read (line => $qr_nlnl, sub {
372 for ("$_[1]\012") { 409 for ("$_[1]\012") {
410 y/\015//d; # weed out any \015, as they show up in the weirdest of places.
411
373 # we support spaces in field names, as lotus domino 412 # we support spaces in field names, as lotus domino
374 # creates them. 413 # creates them (actually spaces around seperators
414 # are strictly allowed in http, they are a security issue).
375 $hdr{lc $1} .= "\x00$2" 415 $hdr{lc $1} .= ",$2"
376 while /\G 416 while /\G
377 ([^:\000-\037]+): 417 ([^:\000-\037]+):
378 [\011\040]* 418 [\011\040]*
379 ((?: [^\015\012]+ | \015?\012[\011\040] )*) 419 ((?: [^\012]+ | \012[\011\040] )*)
380 \015?\012 420 \012
381 /gxc; 421 /gxc;
382 422
383 /\G$/ 423 /\G$/
384 or return (%state = (), $cb->(undef, { Status => 599, Reason => "garbled response headers", URL => $url })); 424 or return (%state = (), $cb->(undef, { Status => 599, Reason => "garbled response headers", URL => $url }));
385 } 425 }
386 426
387 substr $_, 0, 1, "" 427 substr $_, 0, 1, ""
388 for values %hdr; 428 for values %hdr;
389 429
390 my $finish = sub { 430 my $finish = sub {
431 $state{handle}->destroy;
391 %state = (); 432 %state = ();
392 433
393 # set-cookie processing 434 # set-cookie processing
394 if ($arg{cookie_jar} && exists $hdr{"set-cookie"}) { 435 if ($arg{cookie_jar}) {
395 for (split /\x00/, $hdr{"set-cookie"}) { 436 for ($hdr{"set-cookie"}) {
396 my ($cookie, @arg) = split /;\s*/; 437 # parse NAME=VALUE
397 my ($name, $value) = split /=/, $cookie, 2; 438 my @kv;
398 my %kv = (value => $value, map { split /=/, $_, 2 } @arg); 439
399 440 while (/\G\s* ([^=;,[:space:]]+) \s*=\s* (?: "((?:[^\\"]+|\\.)*)" | ([^=;,[:space:]]*) )/gcxs) {
400 my $cdom = (delete $kv{domain}) || $uhost; 441 my $name = $1;
442 my $value = $3;
443
444 unless ($value) {
445 $value = $2;
446 $value =~ s/\\(.)/$1/gs;
447 }
448
449 push @kv, $name => $value;
450
451 last unless /\G\s*;/gc;
452 }
453
454 last unless @kv;
455
456 my $name = shift @kv;
457 my %kv = (value => shift @kv, @kv);
458
459 my $cdom;
401 my $cpath = (delete $kv{path}) || "/"; 460 my $cpath = (delete $kv{path}) || "/";
402 461
462 if (exists $kv{domain}) {
463 $cdom = delete $kv{domain};
464
403 $cdom =~ s/^.?/./; # make sure it starts with a "." 465 $cdom =~ s/^\.?/./; # make sure it starts with a "."
404 466
405 next if $cdom =~ /\.$/; 467 next if $cdom =~ /\.$/;
406 468
407 # this is not rfc-like and not netscape-like. go figure. 469 # this is not rfc-like and not netscape-like. go figure.
408 my $ndots = $cdom =~ y/.//; 470 my $ndots = $cdom =~ y/.//;
409 next if $ndots < ($cdom =~ /\.[^.][^.]\.[^.][^.]$/ ? 3 : 2); 471 next if $ndots < ($cdom =~ /\.[^.][^.]\.[^.][^.]$/ ? 3 : 2);
410 472 } else {
473 $cdom = $uhost;
474 }
475
411 # store it 476 # store it
412 $arg{cookie_jar}{version} = 1; 477 $arg{cookie_jar}{version} = 1;
413 $arg{cookie_jar}{$cdom}{$cpath}{$name} = \%kv; 478 $arg{cookie_jar}{$cdom}{$cpath}{$name} = \%kv;
479
480 redo if /\G\s*,/gc;
481 }
482 }
483
484 # microsoft and other shitheads don't give a shit for following standards,
485 # try to support some common forms of broken Location headers.
486 if ($_[1]{location} !~ /^(?: $ | [^:\/?\#]+ : )/x) {
487 $_[1]{location} =~ s/^\.\/+//;
488
489 my $url = "$rscheme://$uhost:$uport";
490
491 unless ($_[1]{location} =~ s/^\///) {
492 $url .= $upath;
493 $url =~ s/\/[^\/]*$//;
494 }
495
496 $_[1]{location} = "$url/$_[1]{location}";
497 }
498
499 if ($_[1]{Status} =~ /^30[12]$/ && $recurse && $method ne "POST") {
500 # apparently, mozilla et al. just change POST to GET here
501 # more research is needed before we do the same
502 http_request ($method, $_[1]{location}, %arg, recurse => $recurse - 1, $cb);
503 } elsif ($_[1]{Status} == 303 && $recurse) {
504 # even http/1.1 is unclear on how to mutate the method
505 $method = "GET" unless $method eq "HEAD";
506 http_request ($method => $_[1]{location}, %arg, recurse => $recurse - 1, $cb);
507 } elsif ($_[1]{Status} == 307 && $recurse && $method =~ /^(?:GET|HEAD)$/) {
508 http_request ($method => $_[1]{location}, %arg, recurse => $recurse - 1, $cb);
509 } else {
510 $cb->($_[0], $_[1]);
511 }
512 };
513
514 if ($hdr{Status} =~ /^(?:1..|204|304)$/ or $method eq "HEAD") {
515 $finish->(undef, \%hdr);
516 } else {
517 if (exists $hdr{"content-length"}) {
518 $_[0]->unshift_read (chunk => $hdr{"content-length"}, sub {
519 # could cache persistent connection now
520 if ($hdr{connection} =~ /\bkeep-alive\b/i) {
521 # but we don't, due to misdesigns, this is annoyingly complex
522 };
523
524 $finish->($_[1], \%hdr);
525 });
526 } else {
527 # too bad, need to read until we get an error or EOF,
528 # no way to detect winged data.
529 $_[0]->on_error (sub {
530 # delete ought to be more efficient, as we would have to make
531 # a copy otherwise as $_[0] gets destroyed.
532 $finish->(delete $_[0]{rbuf}, \%hdr);
533 });
534 $_[0]->on_eof (undef);
535 $_[0]->on_read (sub { });
414 } 536 }
415 } 537 }
416
417 # microsoft and other shitheads don't give a shit for following standards,
418 # try to support some common forms of broken Location headers.
419 if ($_[1]{location} !~ /^(?: $ | [^:\/?\#]+ : )/x) {
420 $_[1]{location} =~ s/^\.\/+//;
421
422 my $url = "$scheme://$uhost:$uport";
423
424 unless ($_[1]{location} =~ s/^\///) {
425 $url .= $upath;
426 $url =~ s/\/[^\/]*$//;
427 }
428
429 $_[1]{location} = "$url/$_[1]{location}";
430 }
431
432 if ($_[1]{Status} =~ /^30[12]$/ && $recurse && $method ne "POST") {
433 # apparently, mozilla et al. just change POST to GET here
434 # more research is needed before we do the same
435 http_request ($method, $_[1]{location}, %arg, recurse => $recurse - 1, $cb);
436 } elsif ($_[1]{Status} == 303 && $recurse) {
437 # even http/1.1 is unlear on how to mutate the method
438 $method = "GET" unless $method eq "HEAD";
439 http_request ($method => $_[1]{location}, %arg, recurse => $recurse - 1, $cb);
440 } elsif ($_[1]{Status} == 307 && $recurse && $method =~ /^(?:GET|HEAD)$/) {
441 http_request ($method => $_[1]{location}, %arg, recurse => $recurse - 1, $cb);
442 } else {
443 $cb->($_[0], $_[1]);
444 }
445 }; 538 });
539 });
540 };
446 541
447 if ($hdr{Status} =~ /^(?:1..|204|304)$/ or $method eq "HEAD") { 542 # now handle proxy-CONNECT method
448 $finish->(undef, \%hdr); 543 if ($proxy && $uscheme eq "https") {
544 # oh dear, we have to wrap it into a connect request
545
546 # maybe re-use $uauthority with patched port?
547 $state{handle}->push_write ("CONNECT $uhost:$uport HTTP/1.0\015\012Host: $uhost\015\012\015\012");
548 $state{handle}->push_read (line => $qr_nlnl, sub {
549 $_[1] =~ /^HTTP\/([0-9\.]+) \s+ ([0-9]{3}) (?: \s+ ([^\015\012]*) )?/ix
550 or return (%state = (), $cb->(undef, { Status => 599, Reason => "invalid proxy connect response ($_[1])", URL => $url }));
551
552 if ($2 == 200) {
553 $rpath = $upath;
554 &$handle_actual_request;
449 } else { 555 } else {
450 if (exists $hdr{"content-length"}) { 556 %state = ();
451 $_[0]->unshift_read (chunk => $hdr{"content-length"}, sub { 557 $cb->(undef, { Status => $2, Reason => $3, URL => $url });
452 # could cache persistent connection now
453 if ($hdr{connection} =~ /\bkeep-alive\b/i) {
454 # but we don't, due to misdesigns, this is annoyingly complex
455 };
456
457 $finish->($_[1], \%hdr);
458 });
459 } else {
460 # too bad, need to read until we get an error or EOF,
461 # no way to detect winged data.
462 $_[0]->on_error (sub {
463 $finish->($_[0]{rbuf}, \%hdr);
464 });
465 $_[0]->on_eof (undef);
466 $_[0]->on_read (sub { });
467 }
468 } 558 }
469 }); 559 });
560 } else {
561 &$handle_actual_request;
470 }); 562 }
563
471 }, sub { 564 }, sub {
472 $timeout 565 $timeout
473 }; 566 };
474 }; 567 };
475 568
… …
548=head1 AUTHOR 641=head1 AUTHOR
549 642
550 Marc Lehmann <schmorp@schmorp.de> 643 Marc Lehmann <schmorp@schmorp.de>
551 http://home.schmorp.de/ 644 http://home.schmorp.de/
552 645
646With many thanks to Дмитрий Шалашов, who provided countless
647testcases and bugreports.
648
553=cut 649=cut
554 650
5551 6511
556 652

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines