Merge pull request #1 from dustinsmith1024/reconnect-fix

Reconnect fix
auth0 · Jan 15, 2016 · 77e024f · 77e024f
2 parents 8a056d5 + af0ec14
commit 77e024f
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/lib/LdapLookup.js b/lib/LdapLookup.js
@@ -7,11 +7,12 @@ var LdapLookup = module.exports = function(options){
     '(&(objectclass=user)(|(sAMAccountName={0})(UserPrincipalName={0})))';
 
   this._client = options.client ? options.client : ldap.createClient({
-    url:            options.url,
-    maxConnections: 10,
-    bindDN:         options.bindDN,
-    credentials:    options.bindCredentials,
-    tlsOptions:     options.tlsOptions
+    url:             options.url,
+    maxConnections:  options.maxConnections || 10,
+    bindDN:          options.bindDN,
+    bindCredentials: options.bindCredentials,
+    tlsOptions:      options.tlsOptions,
+    reconnect:       options.reconnect
   });
 
   this._client.on('error', function(e){

diff --git a/lib/LdapValidator.js b/lib/LdapValidator.js
@@ -21,6 +21,9 @@ LdapValidator.prototype.validate = function (username, password, callback) {
 
     var client = this._options.binder || ldap.createClient({ url: this._options.url, tlsOptions: this._options.tlsOptions });
 
+    client.on('error', function(e){
+        console.log("Error in LdapValidator", e);
+    })
     //try bind by password
     client.bind(up.dn, password, function(err) {
       if(err) return callback();

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "passport-windowsauth",
-  "version": "0.5.1",
+  "version": "0.6.0",
   "description": "Passport strategy for Windows Integrated Authentication (NTLM)",
   "main": "lib/strategy.js",
   "scripts": {