-
Notifications
You must be signed in to change notification settings - Fork 14
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
Tls client certificate #191
Conversation
0d3438d
to
14a845a
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #191 +/- ##
==========================================
+ Coverage 86.83% 88.21% +1.37%
==========================================
Files 68 68
Lines 5501 5625 +124
==========================================
+ Hits 4777 4962 +185
+ Misses 724 663 -61
☔ View full report in Codecov by Sentry. |
14a845a
to
8166fb4
Compare
8166fb4
to
e986803
Compare
Let's wait for this #185 to be integrated in order to write a couple of tests here before integrating. |
test to validate the TLS connection. Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
with the wrong certificate Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
@DanielePalaia @wolf4ood ready for review. |
test Recreate the certificates Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
test Recreate the certificates Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR add support for client certifications and the possibility to trust the certificates with a trust_certificates method.
I will add tests when #185 will be ready.