httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r678944 - /httpd/httpd/branches/2.2.x/STATUS
Date Tue, 22 Jul 2008 23:34:10 GMT
Author: covener
Date: Tue Jul 22 16:34:09 2008
New Revision: 678944

remove a backport proposal, alternative/backportable solution on trunk coming


Modified: httpd/httpd/branches/2.2.x/STATUS
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Tue Jul 22 16:34:09 2008
@@ -130,31 +130,6 @@
    +1: rpluem, niq
- * mod_auth_digest: Detect during startup when AuthDigestProvider
-   is configured to use an incompatible provider via AuthnProviderAlias.
-   PR 45196 
-   Trunk version of patches:
- (MMN comment)
- (include http_config.h in mod_auth.h)
-   Backport version for 2.2.x of patch:
-   +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
    original behaviour (call access control hooks for internal requests

View raw message