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

[rp...@apache.org]

Author: rpluem
Date: Sat Jul 19 11:49:37 2008
New Revision: 678200

URL: http://svn.apache.org/viewvc?rev=678200&view=rev
Log:
* Add comment

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=678200&r1=678199&r2=678200&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Sat Jul 19 11:49:37 2008
@@ -148,11 +148,20 @@
         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