Drop ENV GITHUB_TOKEN="" from the Dockerfile - #26
Conversation
The empty default tripped the SecretsUsedInArgOrEnv build check that docker/build-push-action@v7 surfaces. GitHubService treats an unset token the same as an empty one, so behavior is unchanged; CONFIG.md now lists the image default as unset. Closes #25 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Self-review rubric (performed inline in the same session; not an independent review):
Dockerfile:25 — behavior note: Out-of-diff observation: both jobs carry a notice that Do-not-auto-merge paths: none matched (no This review was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener). drafted by Claude on behalf of Daniel Stephenson |
Summary
ENV GITHUB_TOKEN=""line is removed fromDockerfile. It tripped theSecretsUsedInArgOrEnvDockerfile build check thatdocker/build-push-action@v7surfaces, because the check matches on the variable name.GitHubServicereads the token withSystem.getenv("GITHUB_TOKEN")and guards ontoken != null && !token.isEmpty(), so an unset variable takes the same anonymous-access path as an empty one.docker-entrypoint.shnever referencesGITHUB_TOKEN, and the token is still supplied at container start bydocker-compose.yml(GITHUB_TOKEN=${GITHUB_TOKEN:-}) and.env.GITHUB_TOKENrow of the Docker environment variables table inCONFIG.mdnow lists the image default as (unset) instead of (empty), matching theTRACE_USAGE_REPORTING/DO_NOT_TRACKrows.CHANGELOG.md[Unreleased]→ Fixed entry records the change.Closes #25
Test plan
mvn testlocally — 84 tests run, 0 failures, 0 errors, 1 skipped (no Java source changed)buildjob greendocker-buildjob green (builds the image and runsdocker-entrypoint-test.sh), with noSecretsUsedInArgOrEnvannotation on the runDeferred issues
LICENSE) was not picked: it needs a copyright holder and year chosen by the maintainer.This PR description was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).
🤖 Generated with Claude Code
drafted by Claude on behalf of Daniel Stephenson