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
signature.asc
Description: Digital signature