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)



-- 
Eric Covener
[EMAIL PROTECTED]

Reply via email to