New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix signatures of \ldap_bind() and \ldap_bind_ext() #2122

Merged

muglug merged 1 commit into vimeo:master from LeSuisse:fix-signature-ldap-bind

Sep 11, 2019

Contributor

LeSuisse commented Sep 11, 2019

Both functions accepts a null $bind_rdn parameter to do an
anonymous bind and they also accept a fourth optional
parameter.

References:
https://www.php.net/manual/en/function.ldap-bind.php
https://github.com/php/php-src/blob/php-7.3.9/ext/ldap/ldap.c#L1152
https://github.com/php/php-src/blob/php-7.3.9/ext/ldap/ldap.c#L1209


          Fix signatures of \ldap_bind() and \ldap_bind_ext()

aeb19ec

Both functions accepts a null $bind_rdn parameter to do an
anonymous bind and they also accept a fourth optional
parameter.

References:
https://www.php.net/manual/en/function.ldap-bind.php
https://github.com/php/php-src/blob/php-7.3.9/ext/ldap/ldap.c#L1152
https://github.com/php/php-src/blob/php-7.3.9/ext/ldap/ldap.c#L1209

muglug merged commit a8e58b0 into vimeo:master

Collaborator

muglug commented Sep 11, 2019

Thanks!

LeSuisse deleted the fix-signature-ldap-bind branch

September 11, 2019 16:05

2e3s pushed a commit to 2e3s/psalm that referenced this pull request


          Fix signatures of \ldap_bind() and \ldap_bind_ext() (vimeo#2122)

069cdb2

Both functions accepts a null $bind_rdn parameter to do an
anonymous bind and they also accept a fourth optional
parameter.

References:
https://www.php.net/manual/en/function.ldap-bind.php
https://github.com/php/php-src/blob/php-7.3.9/ext/ldap/ldap.c#L1152
https://github.com/php/php-src/blob/php-7.3.9/ext/ldap/ldap.c#L1209

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