]> granicus.if.org Git - apache/commitdiff
Vote and promote mod_auth_form fix (and remove duplicate).
authorYann Ylavic <ylavic@apache.org>
Wed, 9 Apr 2014 20:13:53 +0000 (20:13 +0000)
committerYann Ylavic <ylavic@apache.org>
Wed, 9 Apr 2014 20:13:53 +0000 (20:13 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1586125 13f79535-47bb-0310-9956-ffa450edef68

STATUS

diff --git a/STATUS b/STATUS
index ee39ab05bbbaaec5b9ae676addd25d68e48f0b89..4fc6dcf53426db0517e95c832090eb539bafbb24 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -106,16 +106,18 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
   [ New proposals should be added at the end of the list ]
 
   * mod_auth_form: Make sure the optional functions are loaded even when
-    the AuthFormProvider isn't specified.
+    the AuthFormProvider isn't specified. PR 56351.
     trunk patch: https://svn.apache.org/r1531672
     2.4.x patch: trunk patch works, modulo CHANGES
-    +1: minfrin, jim
+    +1: minfrin, jim, ylavic
     -0 covener: does that mean just loading mod_auth_form w/o mod_request 
                 and mod_filter now causes a post-config failure?
     minfrin: It does, however it would only affect configs where mod_auth_form
              has been loaded but not used, as any attempt to use mod_auth_form
              without mod_request and mod_session present would cause the very
              same failure. Would a warning in the release notes be good enough?
+    ylavic: It fixes a runtime crash in this situation (PR 56351), I think
+            post_config (startup) is the best choice => +1
  
    * mod_slotmem_shm: Remove wonky and dangerous code that knows about APR
      internals. We work fine w/o it anyway at this stage.
@@ -251,14 +253,6 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
        The patch includes generated doc changes in order to carry eol-style.
      +1: trawick
 
-   * mod_auth_form: Make sure the optional functions are loaded even when
-                    the AuthFormProvider isn't specified. PR 56351.
-     Submitted/Commited by: minfrin
-     trunk patch: http://svn.apache.org/r1531672
-                  http://svn.apache.org/r1585435
-     2.4.x patch: trunk works (modulo CHANGES)
-     +1: ylavic
-
    * mod_proxy_fcgi: Fix for oddity with mod_proxy_fcgi which can bork
      response headers for mod_deflate-ed responses.
      trunk patch? http://svn.apache.org/r1585824