-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 host name verification #2475
Conversation
8e1a36c
to
fa723a7
Compare
// certificate. | ||
sock.set_verify_mode(ssl::verify_peer); | ||
sock.set_verify_callback(ssl::rfc2818_verification(physicalAddress)); | ||
sock.handshake(ssl_socket::client); |
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.
is it possible to add unit-test? we can reuse certificates used in java-test
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.
I will add unit test first I need to fix the build
8f22bc6
to
2371fb2
Compare
2371fb2
to
bf52dc6
Compare
bf52dc6
to
284aa77
Compare
Added Tests |
ff1ba4a
to
05afabc
Compare
05afabc
to
c9dbfe2
Compare
c9dbfe2
to
f222ab1
Compare
@merlimat @ivankelly please review it. |
f222ab1
to
fd7fa17
Compare
@aahmed-se @merlimat hello, i am catching up the function for go client, want to know the progress of this pull request. |
@wolfstudy I need to rebase this on master but I consider it done for now. |
ddbf5bf
to
53ce603
Compare
53ce603
to
b91c777
Compare
This adds ssl host name verification as an optional client configuration for the C/C++ client library
This brings it to parity with the java client