Skip to content

Add cacerts parameter, which can copy existing cacerts into JDK#284

Closed
mdvorak wants to merge 3 commits into
actions:mainfrom
mdvorak:cacerts
Closed

Add cacerts parameter, which can copy existing cacerts into JDK#284
mdvorak wants to merge 3 commits into
actions:mainfrom
mdvorak:cacerts

Conversation

@mdvorak

@mdvorak mdvorak commented Jan 21, 2022

Copy link
Copy Markdown

Description:
In enterprise environment, custom CA truststore is often used. This parameter allows automatic copy of cacerts file into new Java installation.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

There are no e2e test yet, I can add some if you are interested in merging this.

Note that this modifies actual JDK directory. I was worried about tool cache, but it seems that tool is cached (copied) right after extraction, so copying this file into final directory does not modify tool cache at all. It should be safe.

@IvanZosimov IvanZosimov added feature request New feature or request to improve the current logic needs eyes labels Jul 7, 2023
@brunoborges

Copy link
Copy Markdown
Contributor

The cacerts idea may still be useful, but this branch is too outdated to merge safely. Recommendation: decide whether to pursue via a new scoped PR on current main.

@brunoborges

Copy link
Copy Markdown
Contributor

Closing as outdated due to age and merge conflicts. Continuing this proposal in #1035.

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

Labels

feature request New feature or request to improve the current logic needs eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants