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

WIP: Supply cert for client #41

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Tyil
Copy link

@Tyil Tyil commented Dec 17, 2018

This is still a WIP, since there's still a failing test left!

I'm looking to implement certfp functionality into IRC::Client, and after some reading and discussing I've found that this requires support for SSL_CTX_use_certificate_file. This is already available on .listen, but not yet for .connect. This PR intends to make such functionality available for .connect as well.

The command-line version of this option would be -cert, which can be used as follows:

openssl s_client \
    -connect testnet.darenet.org:6697 \
    -servername testnet.darenet.org \
    -cert client-cert.pem

Now, I'm not terribly familiar with this particular module, or ssllib in general, so any feedback would be greatly appreciated.

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