Ticket #1535 (assigned task)

Opened 5 months ago

Last modified 5 months ago

mod_auth ldap rework

Reported by: dev-zero Assigned to: glen (accepted)
Priority: normal Milestone: 1.5.0
Component: mod_auth Version: 1.5.x-svn
Severity: normal Keywords: patch
Cc: Blocking:
Need Feedback: 1

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.

Attachments

lighttpd-1.5.0_pre1992-ldap.patch (15.8 kB) - added by dev-zero on 01/26/2008 01:53:08 PM.

Change History

01/26/2008 01:53:08 PM changed by dev-zero

  • attachment lighttpd-1.5.0_pre1992-ldap.patch added.

01/26/2008 01:55:48 PM changed by dev-zero

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.

01/26/2008 01:58:11 PM changed by dev-zero

  • keywords set to patch.
  • component changed from mod_auth to mod_evasive.

01/26/2008 01:58:21 PM changed by dev-zero

  • component changed from mod_evasive to mod_auth.

01/26/2008 11:59:17 PM changed by glen

  • owner changed from jan to glen.
  • status changed from new to assigned.

01/27/2008 12:02:10 AM changed by glen

Committed revision [2062]

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?

01/28/2008 01:36:35 AM changed by shogsbro

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

01/28/2008 09:27:47 AM changed by glen

  • pending set to 1.

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)

01/28/2008 09:29:25 AM changed by glen

Committed revision [2067] for compile errors when ldap was not enabled

Add/Change #1535 (mod_auth ldap rework)




Change Properties