ok, I did some investigating and I really think that the mod_perl API is troubled wrt auth_type. from what I tell, $r->auth_type returns the per-dir AuthType config, leaving no way to access r->ap_auth_type.

why are both these important? well, if you want to use the mod_perl API to write a custom authen handler (as opposed to using the existing Basic API) you need (well, you ought, anyway) to do what the C auth handlers do:

/* Now that we are done, set the request_rec values so others will know
* who we are.
*/
r->user = (char*)sent_user;
r->ap_auth_type = "Basic";

in my mind, Apache::RequestRec should provide access to the request_rec fields where appropriate - Apache::RequestUtil should do other per-request stuff. so, this would mean moving the current mpxs_Apache__RequestRec_auth_type to mpxs_Apache__RequestUtil_auth_type and implementing a new mpxs_Apache__RequestRec_auth_type that was a farily simple get/set on the request_rec.

the other alternative is to keep RequestRec focused on the request_rec and leave the config to the $r->per_dir_config API I've seen floating around (but haven't used).

is this making sense?

at any rate, once we get this cleared up, I've got a version of Test::Compat::conn_authen.pm ready to go. however, I guess we'll have to change around lots more tests as well :)

--Geoff


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to