From b8c3dce498f3059280fa3e2e2ee89e6feee09e2c Mon Sep 17 00:00:00 2001 From: Ruediger Pluem Date: Sat, 19 Jul 2008 18:49:37 +0000 Subject: [PATCH] * Add comment git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@678200 13f79535-47bb-0310-9956-ffa450edef68 --- STATUS | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) 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 -- 2.47.2