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

Certificate chain support, Apache Commons Compress support and Skipping of INDEX.LIST #133

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

martinhickson
Copy link

This pull requests provides some code changes that may be of use, which is to enable the certchain jarsigner argument, which is useful when signing with hardware tokens. Additionally, it includes a change to skip the INDEX.LIST file, which prior to the change implicitly results in failed jar downloads when using .pack.gz resources only. Furthermore it adds configurable support for the Apache Commons Compress Pack200 algorithm that may be used to reinstate the pack200 feature that has been removed in newer versions of the JDK.

…, which are:

- Incorrect download artifact names when using packed or versioned jars
- Extra attempted downloads over and above declarations in the JNLP file (via the URL classloading mechanism)
@slawekjaranowski
Copy link
Member

I don't see any test ...
Is it possible and reasonable to add an integration test to src/it or at least a unit test.
Test should show what problem you try to resolve and how to reproduce it.

@martinhickson
Copy link
Author

The pull request illustrates one way of incorporating Apache Commons Compress Pack200 support into the plugin given that native Pack200 support has been removed from the newer JDKs. It would be useful for this to be incorporated in some manner. I will look into the unit testing possibilities further.

@slachiewicz slachiewicz marked this pull request as draft December 22, 2024 21:20
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.

4 participants