From aa1f36fc404a7369f87ce01ee943fe0db97daee6 Mon Sep 17 00:00:00 2001
From: Sarah Alsmiller <sarah.alsmiller@hashicorp.com>
Date: Wed, 28 Jun 2023 16:30:53 -0500
Subject: [PATCH 1/2] fix nil pointer exception

---
 control-plane/api-gateway/common/translation.go | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/control-plane/api-gateway/common/translation.go b/control-plane/api-gateway/common/translation.go
index fd69c8601f..55bfb29a2f 100644
--- a/control-plane/api-gateway/common/translation.go
+++ b/control-plane/api-gateway/common/translation.go
@@ -254,14 +254,16 @@ func (t ResourceTranslator) translateHTTPFilters(filters []gwv1beta1.HTTPRouteFi
 	}
 
 	for _, filter := range filters {
-		consulFilter.Remove = append(consulFilter.Remove, filter.RequestHeaderModifier.Remove...)
+		if filter.RequestHeaderModifier != nil {
+			consulFilter.Remove = append(consulFilter.Remove, filter.RequestHeaderModifier.Remove...)
 
-		for _, toAdd := range filter.RequestHeaderModifier.Add {
-			consulFilter.Add[string(toAdd.Name)] = toAdd.Value
-		}
+			for _, toAdd := range filter.RequestHeaderModifier.Add {
+				consulFilter.Add[string(toAdd.Name)] = toAdd.Value
+			}
 
-		for _, toSet := range filter.RequestHeaderModifier.Set {
-			consulFilter.Set[string(toSet.Name)] = toSet.Value
+			for _, toSet := range filter.RequestHeaderModifier.Set {
+				consulFilter.Set[string(toSet.Name)] = toSet.Value
+			}
 		}
 
 		// we drop any path rewrites that are not prefix matches as we don't support those

From 8ed1d688a5cac90204c6ee9796e0644c2c8b1f9d Mon Sep 17 00:00:00 2001
From: Sarah Alsmiller <sarah.alsmiller@hashicorp.com>
Date: Wed, 28 Jun 2023 16:43:20 -0500
Subject: [PATCH 2/2] add unit test

---
 .../api-gateway/common/translation_test.go    | 54 +++++++++++++++++++
 1 file changed, 54 insertions(+)

diff --git a/control-plane/api-gateway/common/translation_test.go b/control-plane/api-gateway/common/translation_test.go
index caa3efbcac..20917151f3 100644
--- a/control-plane/api-gateway/common/translation_test.go
+++ b/control-plane/api-gateway/common/translation_test.go
@@ -1372,3 +1372,57 @@ func generateTestCertificate(t *testing.T, namespace, name string) corev1.Secret
 		},
 	}
 }
+
+func TestResourceTranslator_translateHTTPFilters(t1 *testing.T) {
+	type fields struct {
+		EnableConsulNamespaces bool
+		ConsulDestNamespace    string
+		EnableK8sMirroring     bool
+		MirroringPrefix        string
+		ConsulPartition        string
+		Datacenter             string
+	}
+	type args struct {
+		filters []gwv1beta1.HTTPRouteFilter
+	}
+	tests := []struct {
+		name   string
+		fields fields
+		args   args
+		want   api.HTTPFilters
+	}{
+		{
+			name:   "no httproutemodifier set",
+			fields: fields{},
+			args: args{
+				filters: []gwv1beta1.HTTPRouteFilter{
+					{
+						URLRewrite: &gwv1beta1.HTTPURLRewriteFilter{},
+					},
+				},
+			},
+			want: api.HTTPFilters{
+				Headers: []api.HTTPHeaderFilter{
+					{
+						Add: map[string]string{},
+						Set: map[string]string{},
+					},
+				},
+				URLRewrite: nil,
+			},
+		},
+	}
+	for _, tt := range tests {
+		t1.Run(tt.name, func(t1 *testing.T) {
+			t := ResourceTranslator{
+				EnableConsulNamespaces: tt.fields.EnableConsulNamespaces,
+				ConsulDestNamespace:    tt.fields.ConsulDestNamespace,
+				EnableK8sMirroring:     tt.fields.EnableK8sMirroring,
+				MirroringPrefix:        tt.fields.MirroringPrefix,
+				ConsulPartition:        tt.fields.ConsulPartition,
+				Datacenter:             tt.fields.Datacenter,
+			}
+			assert.Equalf(t1, tt.want, t.translateHTTPFilters(tt.args.filters), "translateHTTPFilters(%v)", tt.args.filters)
+		})
+	}
+}