Skip to content

Clarify ldap_connect/ldap_bind/ldap_set_option behavior#5275

Open
lacatoire wants to merge 1 commit intophp:masterfrom
lacatoire:fix/ldap-connect-bind-clarify
Open

Clarify ldap_connect/ldap_bind/ldap_set_option behavior#5275
lacatoire wants to merge 1 commit intophp:masterfrom
lacatoire:fix/ldap-connect-bind-clarify

Conversation

@lacatoire
Copy link
Contributor

Summary

  • Clarify that ldap_connect() does NOT open a network connection, only validates URI and
    initializes parameters
  • Document that ldap_bind() is where the actual TCP connection is established
  • Add note to ldap_set_option() that TLS options (LDAP_OPT_X_TLS_*) must be set globally
    (with null) before ldap_bind()
  • Document the correct workflow: ldap_connect()ldap_set_option()ldap_bind()

Fixes #3526

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LDAP: ldap_connect()/ldap_bind()/ldap_set_option() is unclear

1 participant