Re: cvs commit: httpd-2.0/server protocol.c

[Greg Stein <gs...@lyra.org>]

On Sat, Sep 29, 2001 at 08:48:59AM -0000, jerenkrantz@apache.org wrote:
> jerenkrantz    01/09/29 01:48:59
> 
>   Modified:    modules/http http_core.c http_protocol.c
>                server   protocol.c
>   Log:
>   Remove the lameo create_req hack and delay the addition of the HTTP_IN
>   filter until after we have read the headers.  This eliminates the status
>   hack that was in http_protocol.c and makes it all around better.
>   
>   server/protocol.c now directly adds HTTP_IN filter - should we create a
>   specific hook for this?  (Could we do this as a post_read_request hook?)
>   I'm not terribly sure, but let's move it down to the lowest possible
>   place in ap_read_request.  We can change this detail later as we see fit.

Don't worry about it. As long as protocol.c deals with HTTP (specifically,
it manages the reading of the headers, among many other things), then there
isn't a problem with adding the HTTP_IN filter.

If somebody wants to make protocol.c "pure", then they're going to have a
lot more to worry about than the HTTP_IN thing.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/