Do not pulish empty relationships (#327)

Fix bug where identity-api attempts to publish an empty relationship Signed-off-by: Bailin He <[email protected]>
infratographer · Nov 5, 2024 · d567de4 · d567de4
1 parent bdec757
commit d567de4
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/internal/events/groups_event.go b/internal/events/groups_event.go
@@ -38,6 +38,10 @@ func (e *Events) AddGroupMembers(ctx context.Context, gid gidx.PrefixedID, subjI
 		)
 	}
 
+	if len(rels) == 0 {
+		return nil
+	}
+
 	return permissions.CreateAuthRelationships(ctx, GroupTopic, gid, rels...)
 }
 
@@ -58,6 +62,10 @@ func (e *Events) RemoveGroupMembers(ctx context.Context, gid gidx.PrefixedID, su
 		)
 	}
 
+	if len(rels) == 0 {
+		return nil
+	}
+
 	return permissions.DeleteAuthRelationships(ctx, GroupTopic, gid, rels...)
 }