[gateway] add replace for https to support wildcard host #249
Conversation
arnelasse
force-pushed
the
allow-wildcard-host
branch
from
e3d32c9 to
4fe57e7
Compare
arnelasse
force-pushed
the
allow-wildcard-host
branch
from
4fe57e7 to
8c81118
Compare
DanielRaapDev
changed the title
|
Ich habe den Titel um "[gateway]" gepräfixt. |
| version: 1.1.0 | ||
| annotations: | ||
| artifacthub.io/changes: | | ||
| - kind: fixed |
There was a problem hiding this comment.
die Changes bitte aktualisieren
| {{- end }} | ||
| {{- range .Values.gateway.https }} | ||
| - name: "https-{{ .host }}" | ||
| - name: "https-{{ .host | replace "*" "wildcard" | replace "_" "" }}" |
There was a problem hiding this comment.
Sollten auch andere Sonderzeichen ersetzt werden?
Muss die Ersetzung hier im Chart passieren oder wäre es besser wenn bei der Verwendung ein eigener Name angegeben werden kann? gateway.https.nameOverride oder so? Das sieht mir schon sehr spezifisch aus. Liegt das sogar an der Implementation des Gateways welche Namen dort erlaubt sind? Aus YAML Sicht ist das einfach nur ein String 🤔
There was a problem hiding this comment.
Ah OK, die Spec schreibt hier vor, dass nur so etwas wie DNS-Namen verwendet werden dürfen:
MaxLength: 253
MinLength: 1
Pattern:^[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*$
Dann sollte das Replace alles für [^-a-z0-9.] ersetzen durch z.B. -.
3 participants
No description provided.