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

cert: include certificate chain in cert command output #568

Closed
wants to merge 2 commits into from
Closed

cert: include certificate chain in cert command output #568

wants to merge 2 commits into from

Conversation

HonzaCholasta
Copy link
Contributor

@HonzaCholasta HonzaCholasta commented Mar 10, 2017

cert: add output file option to cert-request

The certificate returned by cert-request can now be saved to a file in the
CLI using a new --certificate-out option.

cert: include certificate chain in cert command output

Include the full certificate chain in the output of cert-request, cert-show
and cert-find if --chain or --all is specified.

If output file is specified in the CLI together with --chain, the full
certificate chain is written to the file.

https://pagure.io/freeipa/issue/6547

Jan Cholasta added 2 commits March 13, 2017 11:55
The certificate returned by cert-request can now be saved to a file in the
CLI using a new --certificate-out option.

Deprecate --out in cert-show in favor of --certificate-out.

https://pagure.io/freeipa/issue/6547
Include the full certificate chain in the output of cert-request, cert-show
and cert-find if --chain or --all is specified.

If output file is specified in the CLI together with --chain, the full
certificate chain is written to the file.

https://pagure.io/freeipa/issue/6547
@ghost ghost self-assigned this Mar 14, 2017
@ghost ghost added the ack Pull Request approved, can be merged label Mar 14, 2017
@ghost
Copy link

ghost commented Mar 14, 2017

LGTM and works as expected.

@ghost
Copy link

ghost commented Mar 14, 2017

master:

  • c60d9c9 cert: add output file option to cert-request
  • 8ed891c cert: include certificate chain in cert command output

@ghost ghost added the pushed Pull Request has already been pushed label Mar 14, 2017
@ghost ghost closed this Mar 14, 2017
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
1 participant