Return-Path: Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 32925 invoked by uid 500); 19 Jun 2002 15:52:09 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 32912 invoked from network); 19 Jun 2002 15:52:09 -0000 Date: Wed, 19 Jun 2002 08:52:35 -0700 From: Brian Pane Subject: RE: cvs commit: httpd-2.0/modules/filters mod_include.c In-reply-to: <00ad01c2177f$d6d390b0$0a01230a@KOJ> To: dev@httpd.apache.org Message-id: <1024501955.2946.10.camel@localhost> MIME-version: 1.0 X-Mailer: Ximian Evolution 1.0.3 (1.0.3-4) Content-type: text/plain Content-transfer-encoding: 7BIT References: <00ad01c2177f$d6d390b0$0a01230a@KOJ> X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N On Wed, 2002-06-19 at 03:55, Ryan Bloom wrote: > > > As Justin pointed out on IRC, though, most of the logic to setaside > > the brigade is probably unnecessary because there's currently no way > > for that same filter instance to ever get called again with a > different > > brigade, so the incomplete tag won't ever be completed. > > This whole paragraph scares me, can you please explain it? If we have an incomplete SSI tag at the end of an included file, there's no point in trying to set it aside. When the subrequest completes and we return to its parent request, even if the parent request is itself a server-parsed page, it will have a different include_filter context, so it won't see the setaside partial directive. The only case where we'd need to set aside the partial directive is if there are multiple brigades created for the same request. Is there any handler that does that? --Brian