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

RELENG-6332 configure middleware through env #62

Merged

Conversation

tcarmet
Copy link
Contributor

@tcarmet tcarmet commented Aug 5, 2022

UI was throwing invalid input when behind a nginx ingress

Hard to get a picture of how the ingress behave, making
reverse proxy config manageable through env vars.

It was spotted that there was a typo in the config
for the http scheme, which was decisive in finding
the root cause of the issue

@tcarmet tcarmet requested a review from a team as a code owner August 5, 2022 23:53
@tcarmet tcarmet requested a review from a user August 5, 2022 23:53
@bert-e
Copy link
Contributor

bert-e commented Aug 5, 2022

Hello tcarmet,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Status report is not available.

@bert-e
Copy link
Contributor

bert-e commented Aug 5, 2022

Incorrect Jira project

The Jira issue RELENG-6332 specified in the source
branch name, does not belong to project BERTE.

UI was throwing invalid input when behind a nginx ingress

Hard to get a picture of how the ingress behave, making
reverse proxy config manageable through env vars.

It was spotted that there was a typo in the config
for the http scheme, which was decisive in finding
the root cause of the issue
@tcarmet tcarmet force-pushed the bugfix/RELENG-6332-invalid-input-ingress-conf branch from 0b52bef to f5578c0 Compare August 5, 2022 23:54
@tcarmet tcarmet merged commit de1e826 into development/3.6 Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants