Stas Bekman wrote:
> I've added accessors to Group and User directives in httpd.conf, so that
> now you can get that info via:
> 
>   my $user_id  = Apache2::ServerUtil->user_id;
>   my $group_id = Apache2::ServerUtil->group_id;
> 
> the patch is inlined and attached (due to long lines).
> 
> Please let me know if you are happy with this approach or you want it to
> be different. (including the test).

I am not sure why the test is called apache/deamon ?

> Another concern is windows - so if someone can try to build with this
> patch on windows that would be great.

Patch looks good, at least on Linux/OSX it passes all tests 100%.
It certainly would be nice to get feedback for win32 platforms before
checking it in. Otherwise, I think it's a good idea.

--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to