node-ldapauth-fork icon indicating copy to clipboard operation
node-ldapauth-fork copied to clipboard

[Improvement] Make search timeLimit configurable

Open Hulcsa opened this issue 4 years ago • 1 comments

First of all, thank you for this lib, saved me a lot of time!

There is just one thing that I could not configure and causes some issues in my app. Sadly, my LDAP server response time for a search request may take 15-20 seconds sometimes, but the default search time limit in ldapjs is 10 seconds, and I could not find a way to configure this value. Could you make this limit configurable, please?

Like this for example in ldapauth.js

LdapAuth.prototype._search = function(searchBase, options, callback) {
  var self = this;

  options.timeLimit = self.opts.searchTimeLimit;
  ...

Thanks in advance!

Hulcsa avatar Feb 17 '21 12:02 Hulcsa

This might fix the issue https://github.com/vesse/node-ldapauth-fork/pull/102

gugan-guidanz avatar Jun 28 '22 10:06 gugan-guidanz