Re: [PATCH] Enable all WebDAV methods

[Ben Laurie <be...@algroup.co.uk>]

Roy T. Fielding wrote:
> 
> This needs to be tested before being committed.  I need someone else
> to do it since I am leaving in the morning for a conference trip in
> Orlando and won't have net access again til Thursday night.
> 
>    *) Enabled all of the WebDAV method names for use by third-party
>       modules, Limit, and Script directives.  That includes PATCH,
>       PROPFIND, PROPPATCH, MKCOL, COPY, MOVE, LOCK, and UNLOCK.
>       Improved mod_actions.c so that it can use any of the methods
>       defined in httpd.h.  Added ap_method_number_of(method) for
>       getting the internal method number.  [Roy Fielding]
> 
> It also requires an MMN bump.  I could abstract it more by making the
> methods a configurable array of pointers to static method name strings
> and then replacing the functions in http_protocol.c with simple loops
> through the array (the performance difference is negligible provided
> that the array is ordered by method use frequency).  But that can wait.

Looks OK to me, but another 16 methods and we run out of bits... OK, so
we've got some breathing space, but I suppose this is something we
should look at more carefully for 2.0 (or should we rename that Apache
2000? :-).

Not tested.

Cheers,

Ben.

-- 
Ben Laurie            |Phone: +44 (181) 735 0686| Apache Group member
Freelance Consultant  |Fax:   +44 (181) 735 0689|http://www.apache.org/
and Technical Director|Email: ben@algroup.co.uk |
A.L. Digital Ltd,     |Apache-SSL author     http://www.apache-ssl.org/
London, England.      |"Apache: TDG" http://www.ora.com/catalog/apache/