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

[MJARSIGNER-53] - Add support of the "certchain" parameter in JarSigner sign operation #24

Closed
wants to merge 1 commit into from

Conversation

@oleg-nenashev
Copy link

commented Aug 18, 2017

This adds a new optional parameter for "certchain". If the maintainers are fine with the approach, I will add additional tests.

Downstream pull-request: apache/maven-plugins#125

MJARSIGNER-53

@oleg-nenashev

This comment has been minimized.

Copy link
Author

commented Aug 18, 2017

@olamy

This comment has been minimized.

Copy link
Member

commented Aug 18, 2017

easy I will merge that.
Thanks

@olamy

This comment has been minimized.

Copy link
Member

commented Aug 18, 2017

@reviewbybees

This comment has been minimized.

Copy link

commented Aug 18, 2017

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.