This repository has been archived by the owner on Apr 22, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 7.3k
remove non-ssl/tls bindings from node_crypto.cc #601
Labels
Comments
@ry Is this still relevant? If so, can you expand on it a bit? |
I think this is about moving the encryption stuff out. |
@bnoordhuis we don't need to do this anytime soon - it would be nice to separate the SSL stuff from node_crypto.cc from the crypto stuff. |
+1 for this. I'd started on it somewhere before I started travelling but that was aaages ago. |
@sh1mmer would you like to share any updates on this suggestion? |
@ry do you know, can you confirm me if this was already done? |
This almost got it's way into tls_wrap.cc, I think it should be closed now. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
should be made into new external module.
The text was updated successfully, but these errors were encountered: