Skip to content

Commit fa4fb3a

Browse files
authored
NM-82: check for all network user groups on gateways selection (#3611) (#3615)
* check for all network user groups on gateways selection * check network admin policy for admins and superadmins * remvove default user group changes * add network admin grp to admins users
2 parents 48de8ff + b4c225b commit fa4fb3a

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

pro/logic/acls.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -449,6 +449,15 @@ func ListUserPolicies(u models.User) []models.Acl {
449449
func listPoliciesOfUser(user models.User, netID models.NetworkID) []models.Acl {
450450
allAcls := logic.ListAcls()
451451
userAcls := []models.Acl{}
452+
if _, ok := user.UserGroups[globalNetworksAdminGroupID]; ok {
453+
user.UserGroups[GetDefaultNetworkAdminGroupID(netID)] = struct{}{}
454+
}
455+
if _, ok := user.UserGroups[globalNetworksUserGroupID]; ok {
456+
user.UserGroups[GetDefaultNetworkUserGroupID(netID)] = struct{}{}
457+
}
458+
if user.PlatformRoleID == models.AdminRole || user.PlatformRoleID == models.SuperAdminRole {
459+
user.UserGroups[GetDefaultNetworkAdminGroupID(netID)] = struct{}{}
460+
}
452461
for _, acl := range allAcls {
453462
if acl.NetworkID == netID && acl.RuleType == models.UserPolicy {
454463
srcMap := logic.ConvAclTagToValueMap(acl.Src)

pro/logic/user_mgmt.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -729,7 +729,10 @@ func GetUserRAGNodes(user models.User) (gws map[string]models.Node) {
729729
continue
730730
}
731731
if user.PlatformRoleID == models.AdminRole || user.PlatformRoleID == models.SuperAdminRole {
732-
gws[node.ID.String()] = node
732+
if ok, _ := IsUserAllowedToCommunicate(user.UserName, node); ok {
733+
gws[node.ID.String()] = node
734+
continue
735+
}
733736
} else {
734737
// check if user has network role assigned
735738
if roles, ok := user.NetworkRoles[models.NetworkID(node.Network)]; ok && len(roles) > 0 {

0 commit comments

Comments
 (0)