search

DirectoryTree / LdapRecord

A fully-featured LDAP framework.
https://ldaprecord.com
MIT License
500 stars 44 forks source link

Bug 669 - [Bug] ldap_start_tls(): Unable to start TLS: Local error #704

Closed stevebauman closed 5 months ago

stevebauman commented 5 months ago

Closes #669 Closes #703

This PR implements a new secure property on Ldap connections to track whether the LDAP connection is secure. This allows us to determine if the connection has already been upgraded to TLS prior to attempting to upgrade the connection.

Previously, startTLS() could be called several times in the Auth\Guard, due to using isBound() to determine whether to upgrade the connection.