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

Introduce client_extra_certs option to SSLConfig #414

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timon
Copy link

@timon timon commented Dec 17, 2019

Some websites that rely on client certificate authentication reject requests with proper client certificate unless authority chain is presented with the certificate as well.

This PR introduces a new SSLConfig option, client_extra_certs, that allows supplying CA certificates
along with a request.

@timon
Copy link
Author

timon commented Dec 17, 2019

@nahi I believe that a testcase should be added to this to test/test_ssl.rb, but I'm not sure how to test this.

Some websites relying on client certificate authentication reject
requests with proper client certificate unless authority chain is
presented with the certificate as well. New `client_extra_certs` tunable
allows adding CA certificates to the `extra_chain_cert` option of SSL
connection context, thus providing a way of supplying the authority
chain together with request.
@timon timon force-pushed the feature/ssl-extra-certs branch from d9e59fb to d32f056 Compare December 17, 2019 14:59
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.

1 participant