* Add comment

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@678200 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index 36576b5bd6d8f4edc7bd779da94ffa8f1bd1589a..309869afd8f77ea1258d5256ea14c18cfa377919 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -148,11 +148,20 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
         http://svn.apache.org/viewvc?rev=673999&view=rev (include http_config.h in mod_auth.h)
    Backport version for 2.2.x of patch:
         http://people.apache.org/~covener/2.2.x-digest_vs_authnalias-2.diff
-   +1: covener, rpluem
+   +1: covener
    -1: niq: looks like a major MMN bump to me, in that it'll
             break binary compatibility with old compilations
             of authn modules (has_realm_hash being undefined).
             Why not just test get_realm_hash - as in the else clause?
+   rpluem says: Good catch. Revisiting the patch I agree that it cannot
+   be backported this way and would require a major bump. I guess a way
+   out of this would be to increase the AUTHN_PROVIDER_VERSION and enable
+   the code to deal with different provider versions which would also
+   require changes at other sections of the code like add_authn_provider
+   in mod_auth_basic. Furthermore I wouldn't be sure if older authn
+   providers would still compile against the next 2.2.x version with
+   them missing one element in the authn_provider struct. This would be
+   a non starter of course.
 
  * core, authn/z: Avoid calling access control hooks for internal requests
    with configurations which match those of initial request.  Revert to