Merge pull request #70 from uswitch/airship-2080-fix-request-header-c…

…onfig AIRSHIP-2080 Fix names for config properties
uswitch · Nov 29, 2022 · d725353 · d725353
2 parents 01bd28d + addbc8b
commit d725353
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/pkg/envoy/boilerplate.go b/pkg/envoy/boilerplate.go
@@ -167,8 +167,8 @@ func makeGrpcLoggerConfig(cfg HttpGrpcLogger) *gal.HttpGrpcAccessLogConfig {
 			},
 			TransportApiVersion: core.ApiVersion_V3,
 		},
-		AdditionalRequestHeadersToLog:  cfg.AdditionalRequestHeaders,
-		AdditionalResponseHeadersToLog: cfg.AdditionalResponseHeaders,
+		AdditionalRequestHeadersToLog:  cfg.RequestHeaders,
+		AdditionalResponseHeadersToLog: cfg.ResponseHeaders,
 	}
 }
 

diff --git a/pkg/envoy/configurator.go b/pkg/envoy/configurator.go
@@ -39,11 +39,11 @@ type HttpExtAuthz struct {
 }
 
 type HttpGrpcLogger struct {
-	Name                      string        `json:"name"`
-	Cluster                   string        `json:"cluster"`
-	Timeout                   time.Duration `json:"timeout"`
-	AdditionalRequestHeaders  []string      `json:"additionalRequestHeaders"`
-	AdditionalResponseHeaders []string      `json:"additionalResponseHeaders"`
+	Name            string        `json:"name"`
+	Cluster         string        `json:"cluster"`
+	Timeout         time.Duration `json:"timeout"`
+	RequestHeaders  []string      `json:"requestHeaders"`
+	ResponseHeaders []string      `json:"responseHeaders"`
 }
 
 //KubernetesConfigurator takes a given Ingress Class and lister to find only ingresses of that class