[pmwiki-users] Proposal for authuser script addition

[Thomas Lederer]

Hello,

this seems to be a rather self-centered proposal, but maybe others agree.

As an empty password is no password, but LDAP seems to return a bind
with an emtpy password as true (as in valid), you have to test if the
password is empty. (This seems to be a feature from LDAP not a bug, i
have been told.)

Therefore i would like to see as line 106 in authuser.php:

105> if (!function_exists('ldap_connect')) return false;
106> if (!$pw) return false;
107> foreach ((array)$pwlist as $ldap) {

then i could move it back to scripts folder (from cookbook, where it is
a nuisance to keep updated).

Any other opinions?

Best Regards,
Tom