Skip to content

Commit

Permalink
fix for signature change
Browse files Browse the repository at this point in the history
  • Loading branch information
rboyer committed Sep 22, 2023
1 parent c196e58 commit 06000fa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion internal/mesh/internal/types/destination_policy.go
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ func ValidateDestinationPolicy(res *pbresource.Resource) error {
return merr
}

func aclReadHookDestinationPolicy(authorizer acl.Authorizer, authzContext *acl.AuthorizerContext, id *pbresource.ID) error {
func aclReadHookDestinationPolicy(authorizer acl.Authorizer, authzContext *acl.AuthorizerContext, id *pbresource.ID, _ *pbresource.Resource) error {
// DestinationPolicy is name-aligned with Service
serviceName := id.Name

Expand Down
2 changes: 1 addition & 1 deletion internal/mesh/internal/types/destination_policy_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -574,7 +574,7 @@ func TestDestinationPolicyACLs(t *testing.T) {
authz = acl.NewChainedAuthorizer([]acl.Authorizer{authz, acl.DenyAll()})

t.Run("read", func(t *testing.T) {
err := reg.ACLs.Read(authz, &acl.AuthorizerContext{}, res.Id)
err := reg.ACLs.Read(authz, &acl.AuthorizerContext{}, res.Id, nil)
checkF(t, tc.readOK, err)
})
t.Run("write", func(t *testing.T) {
Expand Down

0 comments on commit 06000fa

Please sign in to comment.