NET-6945 - Replace usage of deprecated Envoy field envoy.config.core.…

…v3.HeaderValueOption.append
hashicorp · Jan 3, 2024 · 554a05f · 554a05f
1 parent d925e4b
commit 554a05f
Show file tree

Hide file tree

Showing 8 changed files with 25 additions and 33 deletions.
diff --git a/agent/xds/routes.go b/agent/xds/routes.go
@@ -534,7 +534,10 @@ func makeHeadersValueOptions(vals map[string]string, add bool) []*envoy_core_v3.
 				Key:   k,
 				Value: v,
 			},
-			Append: response.MakeBoolValue(add),
+		}
+		if !add {
+			// default is APPEND_IF_EXISTS_OR_ADD
+			o.AppendAction = envoy_core_v3.HeaderValueOption_OVERWRITE_IF_EXISTS_OR_ADD
 		}
 		opts = append(opts, o)
 	}
@@ -1130,6 +1133,7 @@ func injectLBToRouteAction(lb *structs.LoadBalancer, action *envoy_route_v3.Rout
 }
 
 func injectHeaderManipToRoute(dest *structs.ServiceRouteDestination, r *envoy_route_v3.Route) error {
+
 	if !dest.RequestHeaders.IsZero() {
 		r.RequestHeadersToAdd = append(
 			r.RequestHeadersToAdd,

diff --git a/agent/xds/testdata/routes/api-gateway-with-http-route-timeoutfilter-one-set.latest.golden b/agent/xds/testdata/routes/api-gateway-with-http-route-timeoutfilter-one-set.latest.golden
@@ -19,14 +19,13 @@
               },
               "requestHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "X-Header-Add",
                     "value": "added"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "X-Header-Set",
                     "value": "set"

diff --git a/agent/xds/testdata/routes/api-gateway-with-http-route.latest.golden b/agent/xds/testdata/routes/api-gateway-with-http-route.latest.golden
@@ -19,14 +19,13 @@
               },
               "requestHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "X-Header-Add",
                     "value": "added"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "X-Header-Set",
                     "value": "set"

diff --git a/agent/xds/testdata/routes/ingress-with-chain-and-router-header-manip.latest.golden b/agent/xds/testdata/routes/ingress-with-chain-and-router-header-manip.latest.golden
@@ -14,14 +14,13 @@
           "name": "db",
           "requestHeadersToAdd": [
             {
-              "append": true,
               "header": {
                 "key": "foo",
                 "value": "bar"
               }
             },
             {
-              "append": false,
+              "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
               "header": {
                 "key": "bar",
                 "value": "baz"
@@ -33,14 +32,13 @@
           ],
           "responseHeadersToAdd": [
             {
-              "append": true,
               "header": {
                 "key": "foo",
                 "value": "bar"
               }
             },
             {
-              "append": false,
+              "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
               "header": {
                 "key": "bar",
                 "value": "baz"
@@ -408,14 +406,13 @@
               },
               "requestHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "request",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"
@@ -427,14 +424,13 @@
               ],
               "responseHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "response",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"

diff --git a/agent/xds/testdata/routes/ingress-with-chain-and-router.latest.golden b/agent/xds/testdata/routes/ingress-with-chain-and-router.latest.golden
@@ -370,14 +370,13 @@
               },
               "requestHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "request",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"
@@ -389,14 +388,13 @@
               ],
               "responseHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "response",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"

diff --git a/agent/xds/testdata/routes/ingress-with-chain-and-splitter.latest.golden b/agent/xds/testdata/routes/ingress-with-chain-and-splitter.latest.golden
@@ -42,7 +42,7 @@
                       "name": "db.default.dc1.internal.11111111-2222-3333-4444-555555555555.consul",
                       "requestHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "db"
@@ -51,7 +51,7 @@
                       ],
                       "responseHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "db"
@@ -64,7 +64,7 @@
                       "name": "big-side.default.dc1.internal.11111111-2222-3333-4444-555555555555.consul",
                       "requestHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "big"
@@ -73,7 +73,7 @@
                       ],
                       "responseHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "big"
@@ -86,7 +86,7 @@
                       "name": "goldilocks-side.default.dc1.internal.11111111-2222-3333-4444-555555555555.consul",
                       "requestHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "goldilocks"
@@ -95,7 +95,7 @@
                       ],
                       "responseHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "goldilocks"
@@ -108,7 +108,7 @@
                       "name": "lil-bit-side.default.dc1.internal.11111111-2222-3333-4444-555555555555.consul",
                       "requestHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "small"
@@ -117,7 +117,7 @@
                       ],
                       "responseHeadersToAdd": [
                         {
-                          "append": false,
+                          "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                           "header": {
                             "key": "x-split-leg",
                             "value": "small"

diff --git a/agent/xds/testdata/routes/xds-fetch-timeout-ms-ingress-with-router.latest.golden b/agent/xds/testdata/routes/xds-fetch-timeout-ms-ingress-with-router.latest.golden
@@ -370,14 +370,13 @@
               },
               "requestHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "request",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"
@@ -389,14 +388,13 @@
               ],
               "responseHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "response",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"

diff --git a/agent/xds/testdata/routes/xds-fetch-timeout-ms-sidecar.latest.golden b/agent/xds/testdata/routes/xds-fetch-timeout-ms-sidecar.latest.golden
@@ -369,14 +369,13 @@
               },
               "requestHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "request",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"
@@ -388,14 +387,13 @@
               ],
               "responseHeadersToAdd": [
                 {
-                  "append": true,
                   "header": {
                     "key": "response",
                     "value": "bar"
                   }
                 },
                 {
-                  "append": false,
+                  "appendAction": "OVERWRITE_IF_EXISTS_OR_ADD",
                   "header": {
                     "key": "bar",
                     "value": "baz"