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

Add support for authentication/search timeout #2

Open

bdpiprava opened this issue Dec 15, 2016 · 0 comments

Contributor

bdpiprava commented Dec 15, 2016 •

edited

Loading

Issue Type

Bug Report

Summary

LdapUserSearch is hard coding the LDAP search timelimit parameter which prevents the administrator from tuning this parameter as appropriate for their environment.

I believe this is normally handled by the system property com.sun.jndi.ldap.read.timeout .

Steps to Reproduce

Find an LDAP server with over 100,000 entries with 'memberOf' overlay enabled
Add an LDAP filter in GoCD to use memberOf
wait 6 seconds

Expected Results

Slow but working response

Actual Results

Soft sobbing

Possible Fix

Remove line 90 in above referenced file

Any other info

I really do like you guys. Ignore the snark.

bdpiprava mentioned this issue

LDAP search timeout shouldn't be hard-coded gocd/gocd#2957

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment