Ingress如不是标准端口，会导致Dify访问不了

[jeessy2]

因为生成的不含端口，如果Ingress访问端口不是443的话，会访问不了。自定义环境变量也不行，会有两个相同的变量名。

charts/charts/dify/templates/_helpers.tpl

Line 76 in e33e9f0

{{- range tuple "CONSOLE_API_URL" "CONSOLE_WEB_URL" "SERVICE_API_URL" "APP_API_URL" "APP_WEB_URL" }}

[LeoQuote]

我这边没这种环境, 你有什么建议的方案?

[LeoQuote]

charts/charts/dify/templates/_helpers.tpl

Lines 64 to 66 in e33e9f0

	{{- define "dify.baseUrl" -}}
	{{ if .Values.global.enableTLS }}https://{{ else }}http://{{ end }}{{.Values.global.host}}
	{{- end }}

这里加一些比如 external port 的 value 如何? 你如果想的话可以直接 pr

[jeessy2]

这样？
如果你在 values.yaml 文件中定义了多个相同名称的环境变量,你可以使用 Helm 的 dict 函数将它们合并为一个单独的 env 列表。

# templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: {{ .Release.Name }}
spec:
  template:
    spec:
      containers:
      - name: {{ .Release.Name }}
        env:
        {{- $envs := dict }}
        {{- range $index, $env := .Values.env }}
        {{- $_ := set $envs $env.name $env }}
        {{- end }}
        {{- range $key, $value := $envs }}
        - name: {{ $key }}
          value: {{ $value.value }}
        {{- end }}

[LeoQuote]

我觉得有点复杂, 我上面提的方案如何呢? 就是加一个 global.port 的value, 如果有就在里面渲染就行

[jeessy2]

也可以

[LeoQuote]

提个 pr 吧, 谢谢~ 你顺便也可以验证下