httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: make_allow
Date Tue, 26 Mar 2002 23:36:57 GMT
On Sun, Mar 24, 2002 at 09:48:24PM -0500, Cliff Woolley wrote:
> 
> There are two problems I see with make_allow() right now.  First, it's
> using method_registry; if ap_method_register() has never been called
> [probably meaning you aren't running DAV], then make_allow() will segfault
> as Brian pointed out.  ap_method_registry_init() needs to be called
> unconditionally from somewhere. I haven't figured out where that is yet.

Server initialization. I believe the pool passed to it is normally pconf
(via post_config). Which basically means that running it after each call to
ap_run_post_config() should be sufficient. That means server/main.c, which
then implies right before ap_mpm_run().

(there are two calls to ap_run_post_config, but there is no need to do it
after the first run, since you're about to wipe the pool anyways)

> Second, the Allow: header is now separated by "," instead of ", " --
> RFC2616 doesn't seem to specify whether that's valid or not, but we should
> probably keep the old behavior.  Unfortunately apr_array_pstrcat (called
> from line 1123 of http_protocol.c) only accepts a single character, not a
> string, as the delimiter.  Ideas?

It's valid behavior (whitespace is allowed in any comma-separated list
unless explicitly specified otherwise).

And empirically: mod_dav has been overriding OPTIONS for a long while now,
and returning a list without spaces.

Cheers,
-g

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

Mime
View raw message