Skip to content

Commit 9180466

Browse files
authored
Merge pull request #53 from ollymaunder-dnv/allow-rewriteUrlPrefix
Add support for uriRegexRewrite.
2 parents bac91f4 + c682e86 commit 9180466

3 files changed

Lines changed: 16 additions & 2 deletions

File tree

charts/platform-service/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ apiVersion: v1
22
appVersion: "1.1"
33
description: Platform service chart
44
name: platform-service
5-
version: 1.4.7
5+
version: 1.5.0

charts/platform-service/templates/_helpers.tpl

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,17 @@ http:
152152
{{- end }}
153153
{{- end }}
154154
{{- if $.Values.defaultRouting.rewriteUrlPrefix.enabled }}
155+
{{- if $.Values.defaultRouting.rewriteUrlPrefix.regexRewrite }}
156+
rewrite:
157+
uriRegexRewrite:
158+
match: {{ required "match is required" $.Values.defaultRouting.rewriteUrlPrefix.regexRewrite.match }}
159+
rewrite: {{ required "rewrite is required" $.Values.defaultRouting.rewriteUrlPrefix.regexRewrite.rewrite }}
160+
{{- else if $.Values.defaultRouting.rewriteUrlPrefix.replaceWith }}
155161
rewrite:
156162
uri: {{ required "rewriteUri is required" $.Values.defaultRouting.rewriteUrlPrefix.replaceWith }}
157-
{{- end}}
163+
{{- else }}
164+
{{- fail "Either replaceWith or regexRewrite are required" }}
165+
{{- end }}
166+
{{- end }}
158167

159168
{{- end -}}

charts/platform-service/values.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,11 @@ defaultRouting:
132132
# alternate prefixes should begin and end with a "/"
133133
# examples: "/", "/api/"
134134
replaceWith: "/"
135+
# Alternatively, use a regex to rewrite the URL path. See https://istio.io/latest/docs/reference/config/networking/virtual-service/#RegexRewrite
136+
# e.g. to rewrite the URL path '/service/update/v1/api' to '/customprefix/v1/api/update', use:
137+
# regexRewrite:
138+
# match: '^\/service\/([^\/]+)(\/.*)$'
139+
# rewrite: '\/customprefix\/\2\/\1'
135140

136141
# Load balancer policy
137142
# Also supported are:

0 commit comments

Comments
 (0)