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

Eliminate dependency on CLI OpenSSL #3

Closed
oxygen opened this issue Sep 26, 2018 · 2 comments
Closed

Eliminate dependency on CLI OpenSSL #3

oxygen opened this issue Sep 26, 2018 · 2 comments

Comments

@oxygen
Copy link

oxygen commented Sep 26, 2018

acme-client requires OpenSSL to be installed and available in $PATH.

https://github.com/digitalbazaar/forge works pretty great. I've used it to replace OpenSSL needs for cross platform nodejs apps which needed certificate management (key generation, CSR and signing).

It would be nice if this library wouldn't depend on CLI tools and it would make it more cross platform than it already is.

If you are open to changing to node forge, I can help with this.

@oxygen oxygen changed the title Eliminate dependency on system tool OpenSSL Eliminate dependency on CLI OpenSSL Sep 26, 2018
@nmorsman
Copy link
Contributor

nmorsman commented Oct 10, 2018

Hey,

Just letting you know that this is in the pipeline and is being worked on right now.

First step will be replacing all internal calls to OpenSSL with the forge equivalent and adding a createPrivateKey() that uses forge. This will likely be shipped with v2.1.0.

OpenSSL will still be kept as an option for backwards compatibility, however it will no longer be required.

I'll keep this open and ping you here once this has been released.

@nmorsman
Copy link
Contributor

nmorsman commented Nov 6, 2018

Just published v2.2.0 with support for node-forge as a cryptographic backend, in addition to the newly added native Node key pair API. All internal calls have also been replaced, removing the OpenSSL CLI requirement.

Documentation is in the README and forge.md.

@nmorsman nmorsman closed this as completed Nov 6, 2018
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

No branches or pull requests

2 participants