On 7/26/2011 5:02 AM, Stefan Fritsch wrote: > This is valid use-case which justifies making it available somehow. > But actually, I think something like > > module *mod = &ap_top_module; > ap_find_command_in_modules(cmd, mod); > > should already work without the need for an additional API. This way > is not very efficient but as this is only done on server startup, > that's not important.
On 7/26/2011 5:35 AM, "Plüm, Rüdiger, VF-Group" wrote: > IMHO the keys in apr_tables are case insensitive. So this tolower shouldn't > be needed. Both points taken and implemented. Regarding invalid directives, I set it as a warning informing that the directive is being discarded. I never actually tested apr_tables to see if they were case sensitive but had assumed they were. The offending lines of code have been removed. Thank you, guys. I attempted to commit this, but ran into the following error. This is my first try to commit, so I very well may be doing something wrong. I'm attempting from my machine here at home rather than minotaur. Any suggestions? svn checkout http://svn.apache.org/repos/asf/httpd/httpd/trunk httpd-trunk && cd http-trunk svn changelist httpd-trunk.AllowOverrideList docs/manual/developer/new_api_2_4.xml docs/manual/mod/core.xml server/config.c server/core.c server/request.c include/http_config.h include/ap_mmn.h include/httpd.h include/http_core.h ... svn commit --username druggeri --password REDACTED --message "Add AllowOverrideList directive and documentation" --changelist httpd-trunk.AllowOverrideList svn: Commit failed (details follow): svn: Server sent unexpected return value (403 Forbidden) in response to MKACTIVITY request for '/repos/asf/!svn/act/2f9005f7-2cd2-4c3f-9311-77bdcf4322e2' User credentials to svn.apache.org web interface and people.apache.org work fine so I suspect there's more to the equation. -- -- Daniel Ruggeri
