You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
mod_auth_openid doesn't seem to be setting the AUTH_TYPE environment variable (which things like basic and digest do).
There is a check in mod_auth_openid.cpp that checks auth_type..
// if we're not enabled for this location/dir, decline doing anything
const char *current_auth = ap_auth_type(r);
if (!current_auth || strcasecmp(current_auth, "openid"))
return DECLINED;
but as far as I can tell... auth_type never gets set. I might be totally wrong, but I'd think you'd want to set auth_type to "openid", and change the test to something like...
mod_auth_openid doesn't seem to be setting the AUTH_TYPE environment variable (which things like basic and digest do).
There is a check in mod_auth_openid.cpp that checks auth_type..
// if we're not enabled for this location/dir, decline doing anything
const char *current_auth = ap_auth_type(r);
if (!current_auth || strcasecmp(current_auth, "openid"))
return DECLINED;
but as far as I can tell... auth_type never gets set. I might be totally wrong, but I'd think you'd want to set auth_type to "openid", and change the test to something like...
if (!current_auth || ( strcasecmp(current_auth, "openid") != 0 ))
return DECLINED;
The text was updated successfully, but these errors were encountered: