vesse / node-ldapauth-fork

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

starttls is insecure: falls back to plaintext #94

Open pcworld opened 3 years ago

pcworld commented 3 years ago

If starttls is used and TLS connection fails, node-ldapauth-fork falls back to plaintext communication, as it does not handle the errors provided by ldapjs properly. This allows an active attacker (or inadvertent misconfiguration) to compromise security.

The library should block connections if starttls fails, and not allow any LDAP commands to be sent in plaintext.