Merge pull request #119 from vivid-planet/fix-multiple-ingress-tls

fix(ingress): Every host needs its own cert for tls
vivid-planet · Dec 18, 2024 · e5dd7ee · e5dd7ee
2 parents d4a0544 + 22a8ad3
commit e5dd7ee
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/.changeset/loud-items-wink.md b/.changeset/loud-items-wink.md
@@ -0,0 +1,5 @@
+---
+"comet-ingress-v1": patch
+---
+
+Every host needs its own cert for tls
diff --git a/charts/comet-ingress-v1/templates/ingress.yaml b/charts/comet-ingress-v1/templates/ingress.yaml
@@ -36,14 +36,16 @@ spec:
             {{ $i.service | toYaml | nindent 12 }}
   {{- end }}
   tls:
-  - hosts:
     {{- if kindIs "slice" $i.hostname }}
-    {{- range $k := $i.hostname }}
-    - {{ $k }}
+    {{- range $j, $k := $i.hostname }}
+    - hosts:
+      - {{ $k }}
+      secretName: {{ $i.service.name }}-{{ $j }}-cert
     {{- end }}
     {{- else }}
-    - {{ $i.hostname }}
+    - hosts:
+      - {{ $i.hostname }}
+      secretName: {{ $i.service.name }}-cert
     {{- end }}
-    secretName: {{ $i.service.name }}-cert
 {{- end }}
 {{- end }}