Skip to content

Commit 8a5ceae

Browse files
committed
update denied_response handling
1 parent 783d0ac commit 8a5ceae

11 files changed

Lines changed: 885 additions & 227 deletions

File tree

internal/xds/bootstrap/bootstrap.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,10 +190,10 @@ type allowedGRPCServiceJSON struct {
190190
// allowedGRPCServicesEnabled reports whether any feature that consumes the
191191
// allowed_grpc_services field is enabled. Per gRFC A102 the field has no
192192
// dedicated env var; it is guarded by the env vars of its consuming features
193-
// (currently only ext_proc on the client; OR in ext_authz/RLQS guards as
193+
// (currently ext_proc and ext_authz on the client; OR in RLQS guards as
194194
// those features are implemented).
195195
func allowedGRPCServicesEnabled() bool {
196-
return envconfig.XDSClientExtProcEnabled
196+
return envconfig.XDSClientExtProcEnabled || envconfig.XDSClientExtAuthzEnabled
197197
}
198198

199199
// AllowedGRPCServices maps a target URI to the credentials gRPC may use for

internal/xds/httpfilter/ext_authz/config.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,16 +20,16 @@ package extauthz
2020

2121
import (
2222
"google.golang.org/grpc/codes"
23+
"google.golang.org/grpc/internal/xds/grpcservice"
2324
"google.golang.org/grpc/internal/xds/httpfilter"
2425
"google.golang.org/grpc/internal/xds/matcher"
25-
"google.golang.org/grpc/internal/xds/xdsclient/xdsresource"
2626
)
2727

2828
// config contains the configuration for the external authorization filter.
2929
type config struct {
3030
httpfilter.FilterConfig
3131
// grpcService is the configuration for the external authorization server.
32-
grpcService xdsresource.GRPCServiceConfig
32+
grpcService *grpcservice.Config
3333
// filterEnabled specifies the percentage of requests to be authorized by
3434
// the external authorization server.
3535
filterEnabled fraction

0 commit comments

Comments
 (0)