From: Ruediger Pluem <rpluem@apache.org>
Date: Sat, 19 Jul 2008 18:49:37 +0000 (+0000)
Subject: * Add comment
X-Git-Tag: 2.2.10~138
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b8c3dce498f3059280fa3e2e2ee89e6feee09e2c;p=thirdparty%2Fapache%2Fhttpd.git

* 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 36576b5bd6d..309869afd8f 100644
--- 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