Open rockycookie opened 1 year ago
Change LGTM, can you add unit tests for the change?
Cool, I will try to find some time next weekend to set up a (or two) unit test(s)~ btw, this code should be working; I have been running some simple load tests with certs with my local build~
added the unit test with self signed client and server certificates at d8a7c57
Hi @rakyll , hope to get a review from you; thx!
Support x509 client cert auth with
-cert
and-cert-key
; while I realized there has been a similar work done at #172 after I made this change lolI would like to see either PR get merged; thx!I would like to merge this PR as I added unit test here lol