Skip to content

Commit df603c5

Browse files
rubenfiszelclaude
andcommitted
Fix windmill-extra WINDMILL_BASE_URL to use http for internal cluster communication
The default was using baseProtocol which could be https when TLS is configured at the ingress, but windmill-app:8000 is a plain HTTP internal service. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent e5f4483 commit df603c5

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

charts/windmill/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v2
22
name: windmill
33
type: application
4-
version: 4.0.93
4+
version: 4.0.94
55
appVersion: 1.650.0
66
dependencies:
77
- condition: minio.enabled

charts/windmill/templates/windmill-extra.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ spec:
6666
- name: REQUIRE_SIGNED_DEBUG_REQUESTS
6767
value: {{ .Values.windmill.windmillExtra.requireSignedDebugRequests | quote }}
6868
- name: WINDMILL_BASE_URL
69-
value: "{{ .Values.windmill.windmillExtra.windmillBaseUrl | default (print .Values.windmill.baseProtocol "://windmill-app:8000") }}"
69+
value: "{{ .Values.windmill.windmillExtra.windmillBaseUrl | default "http://windmill-app:8000" }}"
7070
{{- with .Values.windmill.windmillExtra.extraEnv }}
7171
{{- toYaml . | nindent 8 }}
7272
{{- end }}

charts/windmill/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -558,7 +558,7 @@ windmill:
558558
# -- require signed debug requests (JWT tokens for debug sessions)
559559
requireSignedDebugRequests: true
560560

561-
# -- override WINDMILL_BASE_URL for the extra container (defaults to baseProtocol://windmill-app:8000).
561+
# -- override WINDMILL_BASE_URL for the extra container (defaults to http://windmill-app:8000).
562562
# -- Set to your external URL (e.g. "https://windmill.example.com") if the debugger fails with token verification errors.
563563
windmillBaseUrl: ""
564564

0 commit comments

Comments
 (0)