Skip to content
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

Native TLS as optional #176

Open
mlemesle opened this issue Feb 19, 2023 · 3 comments · May be fixed by #187
Open

Native TLS as optional #176

mlemesle opened this issue Feb 19, 2023 · 3 comments · May be fixed by #187

Comments

@mlemesle
Copy link
Contributor

Hey there !

I'm currently using gremlin-client in one of my project. Unfortunately, gremlin-client rely on native-tls which itself rely on dynamically link openssl.

This can bring errors during compilation (cross compilation) or runtime.

Is it possible to make this dependency optional and rely on rustls instead ?

I might have the time to do it, but I'd like to discuss the implementation with you first !
Please let me know what you think about this !

@wolf4ood
Copy link
Owner

Hi @mlemesle

yes we can definitely, it was the first implementation on the sync connection that relies on native-lts

I can work on this in case since i would also refactor the deps in a way that you can either chose async or sync or both to lower down the dependencies per use case.

@mlemesle
Copy link
Contributor Author

That seems really nice !

Is there any way I can help ?

@mlemesle
Copy link
Contributor Author

Hey there!
Hope eveything's fine!

Did you make some progress on how to implement this?
Feel free to reach if you want to discuss about it or if you need some help for implementation 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants