Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Date: Thu, 7 Mar 2002 01:32:18 -0800
From: Justin Erenkrantz <jerenkrantz@ebuilt.com>
To: dev@httpd.apache.org
Subject: HEADS UP regarding filter names was Re: cvs commit:
 httpd-2.0/server/mpm/perchild perchild.c
Message-ID: <20020307093217.GL24710@ebuilt.com>
Mail-Followup-To: Justin Erenkrantz <jerenkrantz@ebuilt.com>,
	dev@httpd.apache.org
References: <20020307092717.96329.qmail@icarus.apache.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20020307092717.96329.qmail@icarus.apache.org>
User-Agent: Mutt/1.5.0i

On Thu, Mar 07, 2002 at 09:27:17AM -0000, jerenkrantz@apache.org wrote:
> jerenkrantz    02/03/07 01:27:17
> 
>   Modified:    include  ap_mmn.h util_filter.h
>                modules/experimental mod_cache.c mod_case_filter.c
>                         mod_case_filter_in.c mod_charset_lite.c
>                         mod_deflate.c mod_ext_filter.c
>                modules/filters mod_include.c
>                modules/http http_core.c
>                modules/metadata mod_headers.c
>                modules/proxy proxy_ftp.c
>                modules/test mod_bucketeer.c
>                server   core.c util_filter.c
>                server/mpm/perchild perchild.c
>   Log:
>   As hinted on dev@httpd, change filter naming schemes to match our
>   expectations of their usage.

Since we've had so many issues with the filter types recently, I
think it is best to do the rename that we've been hinting at.
This follows from the "three filter types" post from rbb which
gstein and wrowe agreed was goodness (so the idea has been
reviewed).

If you had AP_FTYPE_CONTENT, you should just use AP_FTYPE_RESOURCE.
For those modules that had AP_FTYPE_HTTP_HEADER, you may want
AP_FTYPE_PROTOCOL or AP_FTYPE_CONTENT_SET.  For those filters that
deal with raw protocol issues, they should use _PROTOCOL.  For
those that don't specifically deal with protocols (such as
mod_deflate), they should use _CONTENT_SET.  -- justin