Project

General

Profile

Actions

Feature #1535

closed

mod_auth ldap rework

Added by dev-zero about 16 years ago. Updated almost 8 years ago.

Status:
Fixed
Priority:
Normal
Category:
mod_auth
Target version:
ASK QUESTIONS IN Forums:

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.


Files

Actions #1

Updated by dev-zero about 16 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.

Actions #2

Updated by glen about 16 years ago

  • Status changed from New to Assigned
Actions #3

Updated by glen about 16 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?

Actions #4

Updated by shogsbro about 16 years ago

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

Actions #5

Updated by glen about 16 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)
Actions #6

Updated by glen about 16 years ago

Committed revision r2067 for compile errors when ldap was not enabled

Actions #7

Updated by gstrauss almost 8 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.

Actions

Also available in: Atom