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

Generate CSRs #14

Closed
wants to merge 2 commits into from
Closed

Generate CSRs #14

wants to merge 2 commits into from

Conversation

goetzk
Copy link
Contributor

@goetzk goetzk commented Aug 14, 2017

Hi,
Would you be interested in/willing to merge functionality generate CSRs?
The included code works for no-ops but has a bug when trying to generate the CSR. If you are interested in merging it I will put some effort in to fixing the problem :)

(Commit says its from Sept 2014. It isn't.)
thanks,

When generating a keypair its fairly likely it'll need to be signed by another
CA. This creates the CSR required for that process if openssl_generate_csr is
true.
@franklinkim
Copy link
Member

hi, sure! will be on the road the next couple of days so don't worry if you complete the prs and don't get any from me :)

@franklinkim
Copy link
Member

wanted to say: ... any feedback from me.

- Needed to create the directory for CSRs to be installed in to - OpenSSL
  doesn't do it
- Added the same subject line as was used for generating the certificate to
  suppress questions (which obviously can't be answered).
@goetzk
Copy link
Contributor Author

goetzk commented Aug 16, 2017

I believe this one to be ready, CI permitting :)

@franklinkim
Copy link
Member

THX, merged in release 1.9.0

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

Successfully merging this pull request may close these issues.

2 participants