Skip to content

Commit

Permalink
Merge pull request #17775 from kenrowland/HPCC-30200
Browse files Browse the repository at this point in the history
HPCC-30200 HpccInternal scope should be accessible by HPCC Admins

Reviewed-By: Russ Whitehead <[email protected]>
Merged-by: Gavin Halliday <[email protected]>
  • Loading branch information
ghalliday authored Sep 15, 2023
2 parents f49ad74 + 3f39ca5 commit 1bba010
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion system/security/LdapSecurity/ldapsecurity.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1023,7 +1023,7 @@ SecAccessFlags CLdapSecManager::authorizeFileScope(ISecUser & user, const char *
StringBuffer userName;
for (const char * p = &filescope[m_hpccInternalScope.length()]; *p && *p != ':'; p++)//extract scope username
userName.append(*p);
if(strieq(userName.str(), user.getName()))
if(strieq(userName.str(), user.getName()) || isSuperUser(&user))
return SecAccess_Full;
PROGLOG("Access denied to scope %s for user %s", filescope, user.getName());
return SecAccess_None;
Expand Down

0 comments on commit 1bba010

Please sign in to comment.