Fix Active Logic (#150)

unikorn-cloud · Jan 17, 2025 · d8223a3 · d8223a3
1 parent 5056fdc
commit d8223a3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/charts/identity/Chart.yaml b/charts/identity/Chart.yaml
@@ -4,8 +4,8 @@ description: A Helm chart for deploying Unikorn's IdP
 
 type: application
 
-version: v0.2.51-rc3
-appVersion: v0.2.51-rc3
+version: v0.2.51-rc4
+appVersion: v0.2.51-rc4
 
 icon: https://raw.githubusercontent.com/unikorn-cloud/assets/main/images/logos/dark-on-light/icon.png
 

diff --git a/pkg/rbac/rbac.go b/pkg/rbac/rbac.go
@@ -76,7 +76,7 @@ func (r *RBAC) GetActiveSubjects(ctx context.Context, subject string) (*unikornv
 	}
 
 	result.Items = slices.DeleteFunc(result.Items, func(user unikornv1.User) bool {
-		return user.Spec.Subject != subject && user.Spec.State != unikornv1.UserStateActive
+		return user.Spec.Subject != subject || user.Spec.State != unikornv1.UserStateActive
 	})
 
 	// While we have a list of all user references update their activity status.