Skip to content

Commit 1435d16

Browse files
authored
Merge pull request #20909 from kenrowland/HPCC-35681
HPCC-35681 Review use of logging in LDAP security manager Reviewed-By: Terrence Asselin <terrence.asselin@lexisnexisrisk.com> Merged-by: Gavin Halliday <gavin.halliday@lexisnexisrisk.com>
2 parents 8c6b494 + 03e0672 commit 1435d16

File tree

4 files changed

+17
-158
lines changed

4 files changed

+17
-158
lines changed

system/security/LdapSecurity/aci.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -958,10 +958,7 @@ bool AciProcessor::retrieveUserInfo(ISecUser& user)
958958
CLdapSecUser* ldapuser = (CLdapSecUser*)&user;
959959
const char* username = user.getName();
960960
if(username == NULL || strlen(username) == 0)
961-
{
962-
DBGLOG("AciProcessor::retrieveUserInfo : username is empty");
963961
return false;
964-
}
965962

966963
const char* fullname = user.getFullName();
967964
if((fullname == NULL || *fullname == '\0') && m_ldap_client != NULL)

0 commit comments

Comments
 (0)