We're updating the issue view to help you get more done. 

ldap.readTimeout not used when LDAP getContext() is called for queries

Description

I think I've just found a bug in org.jivesoftware.openfire.ldap.LdapManager - the property ldap.readTimeout is documented at https://www.igniterealtime.org/builds/openfire/docs/latest/documentation/ldap-gu ide.html as "The value of this property is the string representation of an integer representing the read timeout in milliseconds for LDAP operations."

Checking the code at http://fisheye.igniterealtime.org/browse/openfire/trunk/src/java/org/jivesoftwar e/openfire/ldap/LdapManager.java?r=13754 you can see this being applied to the JiveInitialLdapContext environment at line ~656 in the checkAuthentication() method.

However, it is not being applied at all to the getContext() method that starts at line 480.

This means that any LDAP operation that uses getContext() (most of them?) does not have a timeout specified - according to the LDAP Guide listed above, if "no read timeout is specified which is equivalent to waiting for the response infinitely until it is received".

Environment

None

Acceptance Test - Entry

None

Activity

Show:
Greg Thomas
March 20, 2015, 3:34 PM

I've also noticed that the com.sun.jndi.ldap.connect.timeout is only set in checkAuthentication() and not in getContext() - that should probably also be updated, too.

speedy
November 6, 2015, 3:58 PM

submitted pr 364 for review which should add the connection and read time outs to getContext

Assignee

speedy

Reporter

Daryl Herzmann

Labels

None

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Major
Configure