-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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.
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
feature: add timeout to AD auth, default is 5s #6967
Conversation
Naughty powerwalker @murrant :) |
Auto-Deploy finished, Test PR at http://6967.ci.librenms.org or https://6967.ci.librenms.org |
Can be configured by $config['auth_ad_timeout']
Auto-Deploy finished, Test PR at http://6967.ci.librenms.org or https://6967.ci.librenms.org |
Is it easy enough to add a reconnect in so if the connection has timed out it will just auto connect rather than kick the user our (just spit balling) |
Auto-Deploy finished, Test PR at http://6967.ci.librenms.org or https://6967.ci.librenms.org |
The inspection completed: No new issues |
This thread has been automatically locked since there has not been any recent activity after it was closed. |
Can be configured by $config['auth_ad_timeout']
One of my ad servers stop responding to ldap requests and LibreNMS never tried to use the second ad server, just gave a http timeout error.
Adding a timeout allows ldap to give up and try additional servers, or at least actually return some html (which will print an connection failed error).
DO NOT DELETE THIS TEXT
Please note
Testers
If you would like to test this pull request then please run:
./scripts/github-apply <pr_id>
, i.e./scripts/github-apply 5926