Skip to content

Conversation

wey1and
Copy link
Contributor

@wey1and wey1and commented Apr 13, 2022

Closes #180

@wey1and wey1and requested review from ArtDu and vrogach2020 April 13, 2022 09:02
@wey1and wey1and changed the title WIP: Added support SSL/TLS Added support SSL/TLS Apr 14, 2022
@wey1and wey1and requested a review from akudiyar April 14, 2022 16:01
@ArtDu
Copy link
Contributor

ArtDu commented Apr 15, 2022

Maybe can you make tests based on jmh, as I did for mine PR?
#204

We will find out how much slower ssl is compared to regular iproto

@akudiyar
Copy link
Collaborator

Please add a brief description of the changes -- what was changed, why this change is needed.

Copy link
Collaborator

@akudiyar akudiyar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add examples of using the new options and methods in the README and javadocs.

@wey1and wey1and changed the title Added support SSL/TLS WIP: Added support SSL/TLS May 18, 2022
@wey1and wey1and changed the title WIP: Added support SSL/TLS Added support SSL/TLS May 18, 2022
@vrogach2020
Copy link
Contributor

@wey1and please create issues on follow-ups from @akudiyar and @ArtDu

@vrogach2020 vrogach2020 merged commit fd9aff8 into master May 19, 2022
@vrogach2020 vrogach2020 deleted the feature/ssl branch May 19, 2022 10:51
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.

Add TLS/SSL support

4 participants