Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ldapRegistry-3.0 does not configure a read timeout for JNDI connections #3383

Closed
jvanhill opened this issue May 1, 2018 · 0 comments · Fixed by #3635
Closed

ldapRegistry-3.0 does not configure a read timeout for JNDI connections #3383

jvanhill opened this issue May 1, 2018 · 0 comments · Fixed by #3635
Assignees
@jvanhill
Labels
release bug This bug is present in a released version of Open Liberty release-18.0.0.2 team:Wendigo East

Comments

@jvanhill
Copy link
Contributor

jvanhill commented May 1, 2018

The ldapRegistry-3.0 feature does not allow the user to configure, nor does it configure implicitly a read timeout on the JNDI context. This can lead to hangs when an LDAP server does not respond.

Set the read timeout either based on the connect timeout that is already exposed or add a separate read timeout. This value should align closely with the "searchTimeout" used on the SearchControls as the time limit for a search operation.
@jvanhill jvanhill added team:Wendigo East release bug This bug is present in a released version of Open Liberty labels May 1, 2018
@jvanhill jvanhill changed the title ldapRegistry-3.0 configure a read timeout for JNDI connections ldapRegistry-3.0 does not configure a read timeout for JNDI connections May 1, 2018
@jvanhill jvanhill self-assigned this May 18, 2018
@jvanhill jvanhill mentioned this issue May 18, 2018
Merged
jvanhill pushed a commit to jvanhill/open-liberty that referenced this issue May 24, 2018
Issue OpenLiberty#3383: Add LDAP registry JNDI read timeout.
04375a3
jvanhill pushed a commit that referenced this issue May 29, 2018
Merge pull request #3635 from jvanhill/3383-LdapReadTimeout
3eb2e52 
Issue #3383: Add LDAP registry JNDI read timeout.
@jvanhill jvanhill mentioned this issue Sep 4, 2018
Closed
@jvanhill jvanhill mentioned this issue Oct 9, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jvanhill jvanhill

Labels
release bug This bug is present in a released version of Open Liberty release-18.0.0.2 team:Wendigo East
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #3383: Add LDAP registry JNDI read timeout. jvanhill/open-liberty
2 participants
@jvanhill @LibbyBot