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

Additional clusters behind proxy doesn't work #177

Closed
rgeraskin opened this issue Aug 11, 2022 · 3 comments · Fixed by kyverno/policy-reporter-ui#158
Closed

Additional clusters behind proxy doesn't work #177

rgeraskin opened this issue Aug 11, 2022 · 3 comments · Fixed by kyverno/policy-reporter-ui#158

Comments

@rgeraskin
Copy link

Trying to use an amazing feature from #167 I noticed that external clusters behind reverse proxy can't been accessed. E.g. my external policy-reporter is accessed through an nginx ingress.

The reason is

NewSingleHostReverseProxy does not rewrite the Host header. To rewrite Host headers, use ReverseProxy directly with a custom Director policy.

Requests to the ingress controller are submitted with wrong Host header and root policy-reporter can't gather the data. So only non-proxied setups work for now.

@fjogeleit
Copy link
Member

Hey, thanks for your issue. I will check it out.

@fjogeleit
Copy link
Member

fjogeleit commented Aug 11, 2022

I updated the ReverseProxy to also overwrite the request host correctly and add the X-Forwarded-Host and X-Origin-Host header.

I released the version of the UI as v1.6.1

You can try it out by overwriting the Tag in the Helm Values:

ui:
  image:
    tag: 1.6.1

Please let me know if it is working or you still encounter any issues. I will release a new Chart version with this fix next week.

@rgeraskin
Copy link
Author

I've tried it, works like a charm! Thank you

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 a pull request may close this issue.

2 participants