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

Add Support for X-Forwarded-Tls-Client-Cert #12178

Closed
JasonN3 opened this issue Jul 27, 2021 · 4 comments · Fixed by #17272
Closed

Add Support for X-Forwarded-Tls-Client-Cert #12178

JasonN3 opened this issue Jul 27, 2021 · 4 comments · Fixed by #17272
Labels
community-sentiment Tracking high-profile issues from the community enhancement

Comments

@JasonN3
Copy link
Contributor

JasonN3 commented Jul 27, 2021

Is your feature request related to a problem? Please describe.
Currently if you would like to enable certificate authentication, you cannot use SSL offloading since the SSL connection would terminate at the proxy and the only supported X- header is X-Forwarded-For.

Describe the solution you'd like
Add support for the various headers that allow a proxy to include the client cert.
Traefik = X-Forwarded-Tls-Client-Cert
Envoy = X-Forwarded-Client-Cert
F5 = X-Client-Cert

Describe alternatives you've considered
The other option is to disable SSL offloading.

Explain any additional use-cases
This would allow for better integration with various reverse-proxy servers.

@heatherezell heatherezell added auth/cert Authentication - certificates enhancement labels Jul 27, 2021
@nwerker
Copy link

nwerker commented Jan 19, 2022

I would highly appreciate this feature! The best solution would be not only to add a list of supported headers but instead to be able to customize the forwarded headers to support all kind of proxies / load balancers in the future!

@DoktorByte
Copy link

We need exactly this feature for our environment with load balancer and client certificate authentication

@heatherezell heatherezell added community-sentiment Tracking high-profile issues from the community and removed auth/cert Authentication - certificates labels Jan 21, 2022
@ywei2017
Copy link

ywei2017 commented Sep 1, 2022

@hsimon-hashicorp - Any update on this? - accepted/backlogged/planned?

@cwchristerw
Copy link

I would like to be able to use custom header like "X-SSL-Cert" with Nginx Reverse Proxy. I agree that there is need for this feature. I currently can't use cert auth due to this feature missing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community-sentiment Tracking high-profile issues from the community enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants
@cwchristerw @ywei2017 @JasonN3 @nwerker @DoktorByte @heatherezell and others