Skip to content

Commit

Permalink
ocm: don't append IDP to federated user id in ACEs
Browse files Browse the repository at this point in the history
This is no longer needed as the OpaqueID has the idp already included
since 4a02c3e.
  • Loading branch information
rhafer committed Dec 5, 2024
1 parent bddf9f0 commit 6c8fb88
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions pkg/storage/utils/ace/ace.go
Original file line number Diff line number Diff line change
Expand Up @@ -215,10 +215,6 @@ func FromGrant(g *provider.Grant) *ACE {
}

func UserAce(id *userpb.UserId) string {
if id.GetType() == userpb.UserType_USER_TYPE_FEDERATED {
return "u:" + id.OpaqueId + "@" + id.Idp
}

return "u:" + id.OpaqueId
}

Expand Down

0 comments on commit 6c8fb88

Please sign in to comment.