Skip to content

Commit

Permalink
Remove unnecessary log message
Browse files Browse the repository at this point in the history
It is redundant for callers of ldap_connect_bind() to log a message
before executing the function, since it does so already.
  • Loading branch information
dregad committed Jan 27, 2020
1 parent 482ef7a commit f91d60a
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions core/ldap_api.php
Expand Up @@ -212,7 +212,6 @@ function ldap_cache_user_data( $p_username ) {
log_event( LOG_LDAP, "Retrieving data for '$p_username' from LDAP server" );

# Bind
log_event( LOG_LDAP, 'Binding to LDAP server' );
$t_ds = @ldap_connect_bind();
if( $t_ds === false ) {
ldap_log_error( $t_ds );
Expand Down Expand Up @@ -337,7 +336,6 @@ function ldap_authenticate_by_username( $p_username, $p_password ) {
);

# Bind
log_event( LOG_LDAP, 'Binding to LDAP server' );
$t_ds = ldap_connect_bind();
if( $t_ds === false ) {
ldap_log_error( $t_ds );
Expand Down

0 comments on commit f91d60a

Please sign in to comment.