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 11, 2024
1 parent 5a68fdf commit 3e75a1b
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 3e75a1b

Please sign in to comment.