vesse / node-ldapauth-fork

Simple node.js module to authenticate against an LDAP server
Other
126 stars 79 forks source link

Some options have no effect #40

Closed johncylee closed 8 years ago

johncylee commented 8 years ago

Namely,

These are only passed to ldap.createClient from ldapjs, not used anywhere else. However, ldapjs does not use these options.

vesse commented 8 years ago

Good point. Those were used prior to ldapjs 1.0, I didn't properly check the options when I bumped the dependency.