Skip to content

Commit 62ff1db

Browse files
authored
Fix unexpected ";" using zone-sync in the Configmap while disabling IPV6 (#9213)
1 parent 05df869 commit 62ff1db

File tree

3 files changed

+17
-4
lines changed

3 files changed

+17
-4
lines changed

internal/configs/version1/__snapshots__/template_test.snap

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6281,9 +6281,9 @@ http {
62816281
set $resource_namespace "";
62826282
set $service "";
62836283
6284-
listen 0 default_server;listen [::]:0 default_server;
6284+
listen 0 default_server;
62856285
listen 0 ssl default_server;
6286-
listen [::]:0 ssl default_server;
6286+
62876287
ssl_certificate /etc/nginx/secrets/default;
62886288
ssl_certificate_key /etc/nginx/secrets/default;
62896289
@@ -6337,7 +6337,6 @@ stream {
63376337
63386338
server {
63396339
listen 1223;
6340-
listen [::]:1223;
63416340
resolver example.com valid=20s ipv6=off;
63426341
zone_sync;
63436342
zone_sync_server nginx-ingress-headless.nginx-ingress.svc.cluster.local:1223 resolve;

internal/configs/version1/nginx-plus.tmpl

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,9 @@ stream {
353353
{{- if .ZoneSyncConfig.Enable }}
354354
server {
355355
listen {{ .ZoneSyncConfig.Port }};
356-
{{ if not .DisableIPV6}}listen [::]:{{ .ZoneSyncConfig.Port }}{{end}};
356+
{{- if not .DisableIPV6 }}
357+
listen [::]:{{ .ZoneSyncConfig.Port }};
358+
{{- end }}
357359
{{ makeResolver .ZoneSyncConfig.ResolverAddresses .ZoneSyncConfig.ResolverValid .ZoneSyncConfig.ResolverIPV6 }}
358360
zone_sync;
359361
zone_sync_server {{ .ZoneSyncConfig.Domain }}:{{ .ZoneSyncConfig.Port }} resolve;

internal/configs/version1/template_test.go

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1251,12 +1251,23 @@ func TestExecuteTemplate_ForMainForNGINXWithZoneSyncEnabledCustomResolverAddress
12511251
"zone_sync_server nginx-ingress-headless.nginx-ingress.svc.cluster.local:1223 resolve;",
12521252
}
12531253

1254+
unwantedDirectives := []string{
1255+
"listen [::]:1223;",
1256+
}
1257+
12541258
mainConf := buf.String()
12551259
for _, want := range wantDirectives {
12561260
if !strings.Contains(mainConf, want) {
12571261
t.Errorf("want %q in generated config", want)
12581262
}
12591263
}
1264+
1265+
for _, unwanted := range unwantedDirectives {
1266+
if strings.Contains(mainConf, unwanted) {
1267+
t.Errorf("do not want %q in generated config", unwanted)
1268+
}
1269+
}
1270+
12601271
snaps.MatchSnapshot(t, buf.String())
12611272
}
12621273

@@ -2987,6 +2998,7 @@ var (
29872998
}
29882999

29893000
mainCfgWithZoneSyncEnabledCustomResolverAddressAndValidAndIPV6Off = MainConfig{
3001+
DisableIPV6: true,
29903002
ZoneSyncConfig: ZoneSyncConfig{
29913003
Enable: true,
29923004
Port: 1223,

0 commit comments

Comments
 (0)