Return-Path: Delivered-To: apmail-apache-bugdb-archive@apache.org Received: (qmail 74423 invoked by uid 500); 24 Jul 2001 15:25:38 -0000 Mailing-List: contact apache-bugdb-help@apache.org; run by ezmlm Precedence: bulk Reply-To: apache-bugdb@apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list apache-bugdb@apache.org Received: (qmail 74410 invoked from network); 24 Jul 2001 15:25:37 -0000 Received: from masamune.bio.is.tohoku.ac.jp (130.34.233.159) by h31.sny.collab.net with SMTP; 24 Jul 2001 15:25:37 -0000 Received: from dais.is.tohoku.ac.jp (masamune [130.34.233.159]) by masamune.bio.is.tohoku.ac.jp (8.11.3/8.11.3) with SMTP id f6OFPXb07531; Wed, 25 Jul 2001 00:25:33 +0900 (JST) Date: Wed, 25 Jul 2001 00:25:33 +0900 (JST) Message-Id: <200107241525.f6OFPXb07531@masamune.bio.is.tohoku.ac.jp> From: To: stoddard@apache.org Cc: apache-bugdb@apache.org, kabe@sra-tohoku.co.jp Subject: Re: general/8037: [PATCH] HEAD response header repeated twice for files over 32 kilobytes In-Reply-To: Your message of "24 Jul 2001 14:24:44 -0000". <20010724142444.4710.qmail@apache.org> X-Mailer: mnews [version 1.22PL4] 2000-05/28(Sun) X-Spam-Rating: h31.sny.collab.net 1.6.2 0/1000/N >> Thanks for the bug report. I believe the right fix is to uninstall the header filter the first time through on HEAD requests. (right before we return OK). Will be fixed in the next release. Thanks again! >> Bill Thanks, I checked the CVS. (httpd-2.0/modules/http/http_protocol.c 1.330) But I guess this could break in a long run; what if some generator sent down a brigade like 1.Big file bucket 2.Big some bucket 3.EOS Bucket 1 will kick the partial write of the CONTENT_LENGTH, passes it to HEADER and HEADER detaches. Then Bucket 2 is processed by CONTENT_LENGTH, partial write, pass it down; this time CORE gets it, which doesn't care r->header_only and just spits out the content of the bucket to network(bad). I know the above situation won't occur in current Apache2, but some third-party handlers/generators could assemble the brigade. Any thoughts? (I knew goto bad and if(){}ing was possible but indent change makes patch big) -- kabe