Skip to content

Commit 1a9d478

Browse files
committed
fix: Remove backendRefs when using RequestRedirect
Signed-off-by: omar <omar.hammami@solo.io>
1 parent 6bdb48c commit 1a9d478

2 files changed

Lines changed: 7 additions & 16 deletions

File tree

pkg/i2gw/implementations/kgateway/ssl_redirect.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,9 @@ func applySSLRedirectPolicy(
7373
},
7474
},
7575
)
76+
// RequestRedirect filters cannot be used with backendRefs per Gateway API spec
77+
// Remove backendRefs when redirecting
78+
httpRouteContext.Spec.Rules[ruleIdx].BackendRefs = nil
7679
}
7780
}
7881
}

pkg/i2gw/implementations/kgateway/testing/testdata/output/ssl_redirect.yaml

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,7 @@ spec:
3131
parentRefs:
3232
- name: nginx
3333
rules:
34-
- backendRefs:
35-
- name: myservice2
36-
port: 80
37-
filters:
34+
- filters:
3835
- requestRedirect:
3936
scheme: https
4037
statusCode: 301
@@ -59,10 +56,7 @@ spec:
5956
parentRefs:
6057
- name: nginx
6158
rules:
62-
- backendRefs:
63-
- name: myservice
64-
port: 80
65-
filters:
59+
- filters:
6660
- requestRedirect:
6761
scheme: https
6862
statusCode: 301
@@ -71,10 +65,7 @@ spec:
7165
- path:
7266
type: PathPrefix
7367
value: /
74-
- backendRefs:
75-
- name: myservice
76-
port: 80
77-
filters:
68+
- filters:
7869
- requestRedirect:
7970
scheme: https
8071
statusCode: 301
@@ -83,10 +74,7 @@ spec:
8374
- path:
8475
type: PathPrefix
8576
value: /api
86-
- backendRefs:
87-
- name: myservice
88-
port: 80
89-
filters:
77+
- filters:
9078
- requestRedirect:
9179
scheme: https
9280
statusCode: 301

0 commit comments

Comments
 (0)