adelton / mod_authnz_pam

Apache module to run PAM authorization on result of other module's authentication; also full Basic Auth PAM provider.
https://www.adelton.com/apache/mod_authnz_pam/
Apache License 2.0
14 stars 9 forks source link

fix module declaration #16

Closed tessus closed 1 year ago

tessus commented 1 year ago

fixes #15

Update: If you check other modules (if you are allowed to do that - I wasn't when I was working for IBM due to code contamination issues), you'll see that the declaration should be done differently.

There's also Nick Kew's book on Apache modules.

adelton commented 1 year ago

I also filed https://github.com/apache/httpd/pull/359 to get apxs fixed to correctly fallback to guessing the module name based on the filename, as the man page suggests it would do.

tessus commented 1 year ago

It's up to you whether you want to merge this. I think that it certainly wouldn't hurt.

tessus commented 1 year ago

done

adelton commented 1 year ago

Perfect, thanks.