Skip Menu |

This queue is for tickets about the Net-Async-HTTP CPAN distribution.

Report information
The Basics
Id: 79294
Status: resolved
Priority: 0/
Queue: Net-Async-HTTP

People
Owner: Nobody in particular
Requestors: tag [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.17
Fixed in: 0.18



Subject: Support implied keep-alive (http 1.1), keep-alive limits, and rate-limit pipelining
Support implied keep-alive and keep-alive limits. * Added pipelining rate-limiter option, ``max_outstanding'' which reduces the number of concurrently oustanding requests to the value supplied to NaHTTP, tracked on a per-connection basis. * ``max_outstanding'' automatically reduces itself when a server reports a limit on the keep-alive requests allowed, through the ``max'' attribute on the ``keep-alive'' header. * Added ``Connection: keep-alive'' header to outbound requests (by default) to support HTTP/1.0 servers that support the keep-alive extension. * Connections now always assume HTTP/1.1 connections are keep-alive connections unless otherwise reported, as per specification. * Various unit tests updates, they don't quite all work.
Subject: 164.patch

Message body is not shown because it is too large.

Have partially merged: Show quoted text
> * Added ``Connection: keep-alive'' header to outbound requests (by default) > to support HTTP/1.0 servers that support the keep-alive extension.
Show quoted text
> * Connections now always assume HTTP/1.1 connections are keep-alive > connections unless otherwise reported, as per specification.
Not yet done: Show quoted text
> * Added pipelining rate-limiter option, ``max_outstanding'' which reduces the > number of concurrently oustanding requests to the value supplied to NaHTTP, > tracked on a per-connection basis.
Breaks unit tests. Show quoted text
> * ``max_outstanding'' automatically reduces itself when a server reports a > limit on the keep-alive requests allowed, through the ``max'' attribute on > the ``keep-alive'' header.
Depends on above. I'll have a hack on these further and let you know. -- Paul Evans
On Thu Aug 30 16:18:42 2012, PEVANS wrote: Show quoted text
> Not yet done: >
> > * Added pipelining rate-limiter option, ``max_outstanding'' which
reduces the Show quoted text
> > number of concurrently oustanding requests to the value
supplied to NaHTTP, Show quoted text
> > tracked on a per-connection basis.
> > Breaks unit tests.
Now fixed; was an unrelated bug in the continuation handling for "Content-Length: 0" responses. Show quoted text
> > * ``max_outstanding'' automatically reduces itself when a server
reports a Show quoted text
> > limit on the keep-alive requests allowed, through the ``max''
attribute on Show quoted text
> > the ``keep-alive'' header.
> > Depends on above.
Now applied. I believe that's now all in as of upstream -r173 -- Paul Evans
This was released in 0.18. -- Paul Evans