Gisle Aas [EMAIL PROTECTED] wrote:
> Bob McElrath <[EMAIL PROTECTED]> writes:
> 
> > Gisle Aas [EMAIL PROTECTED] wrote:
> > > Bob McElrath <[EMAIL PROTECTED]> writes:
> > > 
> > > > This version doesn't cause a warning if the "Connection" header isn't
> > > > present.
> > > 
> > > Perhaps this warnings shows becase nothing sets 'http_connection'.  
> > 
> > Of course.  The warning was a bug in my original patch.  The patch is
> > trying to fix the transfer-encoding bug indicated below.
> 
> In the source I have there is nothing at all that set or reference
> 'http_connection' that your patch test for.  What version of LWP are
> you patching against?

That is why I checked if it was defined first.  The spec says that not
having a Transfer-Encoding of 'chunked' last is o.k. if there is also a
Connection: close.

'http_connection' should be set by HTTP::Headers, or is there a better
way to get this header in Methods.pm?  (The previous line checked for
'http_te' so i just followed suit...)

> Without the patch I get:
> 
>     [EMAIL PROTECTED] lwp5]$ lwp-request -USe 
> http://larve.net/people/hugo/2000/07/ml-mutt | head -30
>     GET http://larve.net/people/hugo/2000/07/ml-mutt
>     User-Agent: lwp-request/2.01
>     
>     GET http://larve.net/people/hugo/2000/07/ml-mutt --> 200 OK
>     Cache-Control: max-age=21600
>     Connection: close
>     Date: Tue, 14 Oct 2003 16:09:12 GMT
>     ETag: "8buij7:ub1p6un0"
>     Server: Jigsaw/2.2-20010823 jre/1.2.2_009 javacomp/1.2.15
>     ...
> 
> The TCP dump shows this conversation:
> 
>     GET /people/hugo/2000/07/ml-mutt HTTP/1.1
>     TE: deflate,gzip;q=0.3
>     Connection: TE, close
>     Host: larve.net
>     User-Agent: lwp-request/2.01
>     
>     HTTP/1.1 200 OK
>     Cache-Control: max-age=21600
>     Connection: close
>     Date: Tue, 14 Oct 2003 16:09:12 GMT
>     Transfer-Encoding: deflate,chunked
>     Opt: "http://www.w3.org/2000/P3Pv1";ns=11
>     Content-Location: http://larve.net/people/hugo/2000/07/ml-mutt.html
>     Content-Type: text/html;charset=us-ascii
>     Etag: "8buij7:ub1p6un0"
>     Expires: Tue, 14 Oct 2003 22:09:12 GMT
>     Last-Modified: Thu, 16 Jan 2003 04:11:55 GMT
>     Server: Jigsaw/2.2-20010823 jre/1.2.2_009 javacomp/1.2.15
>     11-PolicyRef: /2000/08/p3p-policyref
>     
>     400
>     [binary data]
> 
> Are you perhaps talking via a proxy server of some kind?

Ah...there is a 'transparent' caching proxy server for the campus.  Not
too transparent, eh?

From other hosts not behind the UW proxy I see the same behavior you
see.  LWP 5.65 - 5.69 on debian and redhat.

Nonetheless I think the response I am seeing (after proxy munging) is
valid so shouldn't generate an error.

Cheers,
Bob McElrath [Univ. of California at Davis, Department of Physics]

    "Knowledge will forever govern ignorance, and a people who mean to
    be their own governors, must arm themselves with the power knowledge
    gives. A popular government without popular information or the means
    of acquiring it, is but a prologue to a farce or a tragedy or
    perhaps both."
        - James Madison

Attachment: signature.asc
Description: Digital signature

Reply via email to