> -----Ursprüngliche Nachricht----- > Von: Eric Covener > Gesendet: Montag, 21. Juli 2008 14:09 > An: [email protected] > Betreff: Re: svn commit: r678179 - /httpd/httpd/branches/2.2.x/STATUS > > On Sat, Jul 19, 2008 at 12:03 PM, <[EMAIL PROTECTED]> wrote: > > > * mod_auth_digest: Detect during startup when AuthDigestProvider > > is configured to use an incompatible provider via > AuthnProviderAlias. > > @@ -164,6 +149,10 @@ > > 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: 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? > > Is it kosher to pull the API extension from trunk? A less intrusive > (AuthnProviderAlias-only) solution occurred to me while reading your > note. Would the (incorrect) minor bump be left in place or can it be > reverted? > > (AuthnProviderAlias can just register a different authn_provider > struct and not put a reference to its own own get_realm_hash() in it)
Could you please elaborate this in more detail? Regards Rüdiger
