Return differently for unknown user, allows other authenticators

phokz / mod-auth-external

External Authentication Module for Apache HTTP Server - mod_authnz_external

37 stars 14 forks source link

Return differently for unknown user, allows other authenticators #26

Closed mkj closed 8 years ago

mkj commented 8 years ago

This is an attempt at fixing #3 , it allows using "AuthBasicProvider external file".

Definitely requires review by someone who has more than 15 minutes familiarity with Apache auth providers.

phokz commented 8 years ago

Hi, patch seems to be readable and simple, so it should be merged. I'll try to build it and test, just to be sure. Could you please also write a paragraph with config examples you are using and add it to README?

mkj commented 8 years ago

Looking further I don't think this should be applied in its current form. The exit code of "1" as STATUS_UNKNOWN is specific to pwauth - the only generally defined code for authenticators is 0 SUCCESS.