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

Jump to bottom

draft: TLS support for meta and storage clients #239

Closed

wey-gu wants to merge 3 commits into vesoft-inc:master from wey-gu:ssl_all_clients

Contributor

wey-gu commented Nov 15, 2022

as titled.

wey-gu added 3 commits

November 15, 2022 15:39


          sclient TLS support

ad7640d


          SNI support: SSL_set_tlsext_host_name

cab4492

SSL_set_tlsext_host_name is the openssl function
to add SNI, this is needed in TLSv1.3.

ref:
- https://docs.python.org/3/library/ssl.html#ssl.SSLContext.sslsocket_class
- apache/thrift@937228e
- https://github.com/apache/thrift/blob/e8353cb46e9f5e71f9b76f55d6bf59530b7f98ef/lib/py/src/transport/TSSLSocket.py#L184


          mclient: tls support

730f4ff

wey-gu changed the title ~~TLS support for meta and storage clients~~ draft: TLS support for meta and storage clients

Contributor Author

wey-gu commented Nov 15, 2022

- StorageClient verified manually
- MetaClient verification TBD
- UT

Contributor

Sophie-Xie commented Apr 19, 2023

It's too long. I'll close it and reopen if necessary.

Sophie-Xie closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment