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.

Sign up for GitHub

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

Jump to bottom

[charts] fix url concatenation so we don't get // in the urls #1069

Open
1 task
jbygdell opened this issue Oct 4, 2024 · 0 comments
Open
1 task

[charts] fix url concatenation so we don't get // in the urls #1069

jbygdell opened this issue Oct 4, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@jbygdell
Copy link
Collaborator

jbygdell commented Oct 4, 2024 •
edited
Loading

Describe the bug
Url paths are doen by simply joining the strings which in some cases generates double slashes.
OIDC hostnames needs to end with / or the matching will fail.
Steps to reproduce

Expected behavior

  • charts can be deployed with no issues

Additional context

Estimation of size: small

Estimation of priority: low
@jbygdell jbygdell added the bug Something isn't working label Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jbygdell