George Heeres

Any update on this?

CKarper, is this fix in a github repository that I can extract or pull? Or if you can send me your personal library, I can diff, extract and apply it....

Please apply the patch / change that Damassk specified and get back to us on your results. If confirmed, I can merge those changes and publish and updated version.

Did you try running the example code provided. Also try using a userPrincipalName instead of a sAMAccountName, although either should work. ``` javascript var sAMAccountName = '[email protected]'; var ad =...

Please double check your configuration: ``` javascript var adConfig = { url: 'ldap://domain', baseDN: 'dc=domain,dc=net'} var ad = new activedirectory(adConfig); ad.opts.bindDN = 'domain\username'; ad.opts.bindCredentials = 'password'; ``` The username /...

By default, the group membership information containing the SID is not included. If I recall, you can change the default attributes that are returned for a group to include the...

If I recall, the object sids should automatically be included / converted to strings when the attribute is specified. Just specify the attribute you want into the defaults or specify...

...or even best: update the ActiveDirectory library to support the changes for ldapjs 1.0. :) In the meantime, it looks like you can send an opts object with your query...

See if the following diff/patch fixes the problem: ``` diff --- a/lib/activedirectory.js +++ b/lib/activedirectory.js @@ -1811,6 +1811,7 @@ ActiveDirectory.prototype.authenticate = function authenticate(username, passwor var errorHandled = false; function handleError(err) {...

Reading through the ldapjs source. Trying changing the .unbind call to .destroy. ``` diff --- a/lib/activedirectory.js +++ b/lib/activedirectory.js @@ -1811,6 +1811,7 @@ ActiveDirectory.prototype.authenticate = function authenticate(username, passwor var errorHandled =...