boomerang-io / charts

Helm charts for Boomerang Projects
Apache License 2.0
3 stars 5 forks source link

:bug: `basic-auth-pasword` variable was added inline #4

Closed aiciobanu closed 4 years ago

aiciobanu commented 4 years ago

Closes #

[bmrg-auth-proxy] Authentication fails when using basic-auth mode #3

Changelog

New

Changed

Removed

Testing / Reviewing

helm3 install <release-name> --namespace <auth-namespace> <helm repo / chart package> --set auth.displayHtpasswdForm=true --dry-run

basic-auth-password value should be on its own line:

spec:
  ...
  template:
    ...
    spec:
      containers:
      - args:
        ...
        - --pass-basic-auth=true
        - --basic-auth-password=$(UPSTREAM_AUTH_PASSWORD)
        - --email-domain=*