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

feat(core) support static client SSL certificates #2556

Merged
merged 1 commit into from
May 25, 2017

Conversation

p0pr0ck5
Copy link
Contributor

Summary

Implement the proxy_ssl_certificate and proxy_ssl_certificate_key
directives in the Kong template. Because OpenResty's balancer_by_lua
handler does not cover client side certs, we cannot assign client
certs on a configurable basis, and using complex values in cert
values is unsupported in Nginx, we are left with a static approach
that can only serve one cert/key pair.

Full changelog

  • Add proxy_ssl_certificate and proxy_ssl_certificate_key in Kong template

Issues resolved

Fix #1547

@subnetmarco subnetmarco added this to the 0.10.3 milestone May 24, 2017
@p0pr0ck5 p0pr0ck5 force-pushed the feat/proxy_ssl_certificate branch from e26a24a to 87e0b2a Compare May 25, 2017 00:36
Implement the proxy_ssl_certificate and proxy_ssl_certificate_key
directives in the Kong template. Because OpenResty's balancer_by_lua
handler does not cover client side certs, we cannot assign client
certs on a configurable basis, and using complex values in cert
values is unsupported in Nginx, we are left with a static approach
that can only serve one cert/key pair.
@p0pr0ck5 p0pr0ck5 force-pushed the feat/proxy_ssl_certificate branch from 87e0b2a to e972faa Compare May 25, 2017 00:37
@thibaultcha thibaultcha merged commit fd86e20 into master May 25, 2017
@thibaultcha thibaultcha deleted the feat/proxy_ssl_certificate branch May 25, 2017 00:40
thibaultcha added a commit to Kong/docs.konghq.com that referenced this pull request May 25, 2017
thibaultcha added a commit to Kong/docs.konghq.com that referenced this pull request May 25, 2017
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.

Add configuration support for client-side SSL certs
3 participants