svn commit: r815281 - /httpd/httpd/branches/2.2.x/STATUS

[tr...@apache.org]

Author: trawick
Date: Tue Sep 15 11:37:34 2009
New Revision: 815281

URL: http://svn.apache.org/viewvc?rev=815281&view=rev
Log:
answer a question embedded in a patch

Modified:
    httpd/httpd/branches/2.2.x/STATUS

Modified: httpd/httpd/branches/2.2.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=815281&r1=815280&r2=815281&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Tue Sep 15 11:37:34 2009
@@ -142,7 +142,10 @@
    Trunk patch: n/a due to authz refactoring (no provider called without require-ments)
    2.2.x version of patch: http://people.apache.org/~covener/httpd-2.2.x-authnz_ldap-skipdnloookup-2.diff
    +1: covener, poirier
-
+   trawick: The code for "FIXME: dead code?" is indeed dead, as is the
+            conditional setting of reqs_array within the declarations.  The hook won't
+            be called if ap_requires() returns NULL.  (See calls to ap_some_auth_required()
+            before calls to ap_run_auth_checker().)
    
 PATCHES/ISSUES THAT ARE STALLED