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

Make it easy to customize cnf/crt/csr/key paths #46

Merged
merged 1 commit into from
Sep 23, 2015

Conversation

robbat2
Copy link
Contributor

@robbat2 robbat2 commented May 14, 2015

No description provided.

@cjeanneret
Copy link
Contributor

Hello @robbat2 !

Thank you for your PR, but it seems it breaks the unit tests (especially the second commit). You might have a look here for example:
https://travis-ci.org/camptocamp/puppet-openssl/jobs/62590856#L256

You should update the unit tests in order to ensure the new cnf_* values are taken into account as expected.

Thank you for your update.

Cheers,

C.

@raphink
Copy link
Member

raphink commented Sep 15, 2015

Thank you for this @robbat2 . Could you rebase on master and squash your commits please? And then I'll merge.

Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
@robbat2
Copy link
Contributor Author

robbat2 commented Sep 22, 2015

Rebased+squashed as requested.

@raphink raphink merged commit 57dbe6b into voxpupuli:master Sep 23, 2015
@robbat2 robbat2 deleted the path_config branch September 24, 2015 21:03
@raphink raphink added the enhancement New feature or request label Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants