Project

General

Profile

Feature #1535

mod_auth ldap rework

Added by dev-zero over 11 years ago. Updated about 3 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
Category:
mod_auth
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Missing in 1.5.x:

Description

While looking at the bugs currently present, a rework of the ldap-authentication in mod_auth is needed.
The patch I'll attach not only fixes a couple of bugs, it also makes lighttpd use the new ldap api and makes it possible to start lighttpd even if the ldap-server is not reachable, which looses the service dependencies.

In detail, it fixes the following tickets: #464, #529, #1066, #1090, #1093, #1096

What is needed in addition to this is an update of the documentation and the configuration-file.

History

#1

Updated by dev-zero over 11 years ago

What has changed in the configuration file is that the attribute "hostname" has been replaced by "url".
In addition, I think that this code handles errors better than the original one.

#2

Updated by glen over 11 years ago

  • Status changed from New to Assigned
#3

Updated by glen over 11 years ago

Committed revision r2062

Please don't forget to update the documentation. and perhaps make the old hostname also supported and mark it as deprecated with T_CONFIG_DEPRECATED?

#4

Updated by shogsbro over 11 years ago

This fails to build for me and another person (see tickets #1536 & #1538)

#5

Updated by glen over 11 years ago

how about the backward compatible option?

and please update also config in doc and for tests:


WARNING: unknown config-key: auth.backend.ldap.hostname (ignored)
#6

Updated by glen over 11 years ago

Committed revision r2067 for compile errors when ldap was not enabled

#7

Updated by gstrauss about 3 years ago

  • Description updated (diff)
  • Status changed from Assigned to Fixed

Fixed. Above notes indicate that the patch was committed to lighttpd-1.5.x branch.

Also available in: Atom