Merge pull request #27 from infobloxopen/authorizerOpts

authorizor options were being ignored
infobloxopen · Jun 16, 2023 · d191e93 · d191e93
2 parents 538f3d7 + 657a067
commit d191e93
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/grpc_opa/server_interceptor.go b/grpc_opa/server_interceptor.go
@@ -35,9 +35,13 @@ func NewDefaultConfig(application string, opts ...Option) *Config {
 		opt(cfg)
 	}
 
+	opts = append([]Option{WithOpaClienter(opa_client.New(cfg.address, opa_client.WithHTTPClient(cfg.httpCli)))}, opts...)
+
+	authorizer := NewDefaultAuthorizer(application, opts...)
+
 	if cfg.authorizer == nil {
 		logrus.Info("authorizers empty, using default authorizer")
-		cfg.authorizer = []Authorizer{NewDefaultAuthorizer(application, opts...)}
+		cfg.authorizer = []Authorizer{authorizer}
 	}
 
 	return cfg