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 client and peer certificates #106

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

Conversation

gshuflin
Copy link

This commit adds two additional command-line flags for specifying SSL
certificates to be used by drill when making HTTP requests.

--cert is used to specify a client certificate file (following the
curl option with the same name). Again similarly to curl, --cacert
is used to specify a PEM format certificate that should be used to
verify the remote peer.

This commit adds two additional command-line flags for specifying SSL
certificates to be used by drill when making HTTP requests.

`--cert` is used to specify a client certificate file (following the
`curl` option with the same name). Again similarly to curl, `--cacert`
is used to specify a PEM format certificate that should be used to
verify the remote peer.
@4F2E4A2E
Copy link

This is a quite interesting PR, thank you, for that. For me, it is unclear why exactly the CI [1] is failing. Do you need a hand investigating that?

1: https://app.travis-ci.com/github/fcsonline/drill/jobs/506791339#L724

@corrieriluca
Copy link
Contributor

The CI is failing because the cargo fmt --all -- --check fails, i. e. the formatting of the code is incorrect (here there are trailing new lines at the end of src/config.rs).

@gshuflin
Copy link
Author

Looks like there have been some substantial changes to the existing code since the last time I was looking at it. This is a patch I implemented to solve a particular problem at a company I no longer work at and haven't thought about in a long time, so I'm not in a good position to debug the merge conflicts. If anyone else wants to take this code and create a working patch out of it, they are welcome to do so.

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.

4 participants