Repository navigation
Import Maven signing keys into an isolated GPG home - #1214
Conversation
Import signing keys into an action-owned temporary GPG home, export GNUPGHOME, and remove the owned directory in the post action. Cover import failure, multiple keys and invocations, unrelated keyrings, missing state, and Windows path conversion. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Account for isolated GPG-home cleanup when cache saving is disabled. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Apply repository formatting and commit the setup and cleanup bundles produced by the validated Node 24 build. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates actions/setup-java to import Maven signing keys into an action-owned, isolated temporary GPG home (exported via GNUPGHOME) and to clean up by deleting the owned GPG home directory instead of deleting fingerprints from the runner’s default keyring.
Changes:
- Create a per-invocation temporary GPG home with restrictive permissions and import
gpg-private-keyinto that isolated keyring. - Persist the owned GPG home path as post-action state and remove the directory during cleanup with ownership/safety checks.
- Expand test coverage and update docs/action metadata to reflect isolated key handling; regenerate
dist/bundles.
Show a summary per file
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 12/17 changed files
- Comments generated: 1
- Review effort level: Lite
|
Nice work — this matches how I'd approach #1211 almost exactly (isolated home, 1.
Best-effort kill before removal handles both: try {
await exec.exec(
'gpgconf',
['--homedir', toGpgPath(resolvedGpgHome), '--kill', 'gpg-agent'],
{silent: true, ignoreReturnCode: true}
);
} catch {
// gpgconf may not be present; removal is still attempted
}
await io.rmRF(resolvedGpgHome);2.
The |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Summary
GNUPGHOMEValidation
npm run formatnpm run lintnpm run buildnpm test(41 suites, 1,322 tests)Closes #1211