[Nagiosplug-devel] [ nagiosplug-Patches-836010 ] check_ldap bug fix

[SourceForge.net]

Patches item #836010, was opened at 2003-11-04 20:18
Message generated for change (Comment added) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=836010&group_id=29880

Category: Bugfix
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Howard Wilkinson (howardwilkinson)
Assigned to: Ton Voon (tonvoon)
Summary: check_ldap bug fix

Initial Comment:
These patches were needed to make check_ldap work with
the latest OpenLdap release on our system.

I think they are general purpose but an alternative
assessment would be useful!

----------------------------------------------------------------------

>Comment By: Ton Voon (tonvoon)
Date: 2004-03-19 09:57

Message:
Logged In: YES 
user_id=664364

Howard,

I'm not totally familiar with OpenLDAP. Your patch looks like it 
changes the ldap_open to ldap_init, but would this break 
other ldap implementations?

I'm a bit worried about dropping the -D and -P parameters as 
this breaks any current syntax configuration.

Would it be better to put an if statement around the 
initialization so that if the -D and -P are not passed, then call 
ldap_init, otherwise call ldap_open?

Ton

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=836010&group_id=29880