Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

helm chart shouldn't be check for scheme on redirect #1345

Open
elrondvega opened this issue Feb 20, 2025 · 1 comment
Open

helm chart shouldn't be check for scheme on redirect #1345

elrondvega opened this issue Feb 20, 2025 · 1 comment
Labels
help wanted PRs are welcome on this issue kind/bug/confirmed a confirmed bug (reproducible).

Comments

@elrondvega
Copy link

{{- $errorMsg := printf "ERROR: Cannot redirect %s to %s without setting scheme to https" $entrypoint .to }}

The helm chart shouldn't need to force this value to be set as the default when unset is correct.

@mloiseleur mloiseleur added the kind/bug/confirmed a confirmed bug (reproducible). label Feb 24, 2025
@mloiseleur
Copy link
Member

That's correct @elrondvega.
Feel free to open a PR if you need it.

@mloiseleur mloiseleur added the help wanted PRs are welcome on this issue label Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted PRs are welcome on this issue kind/bug/confirmed a confirmed bug (reproducible).
Projects
None yet
Development

No branches or pull requests

2 participants