Skip to content

Commit 6fa7ea9

Browse files
TeoSlayerteovlclaude
authored
updater: correct --skip-attestation help (verification is now gh-free) (#375)
updater: correct --skip-attestation help — verification is now gh-free The updater module no longer shells out to `gh`; it verifies SLSA provenance in-process via sigstore-go and fails closed. Update the flag help and comment so they no longer say a host without `gh` needs --skip-attestation. Co-authored-by: Teodor Calin <teodor@vulturelabs.io> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 87960d3 commit 6fa7ea9

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

cmd/updater/main.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,13 @@ func main() {
5151
showVersion := flag.Bool("version", false, "print version and exit")
5252
statePath := flag.String("state-path", defaultStatePath(), "JSON control file {\"enabled\":bool} for automatic updates; auto-update is OFF until enabled (e.g. via `pilotctl update enable`)")
5353
// --skip-attestation opts out of SLSA provenance verification of
54-
// checksums.txt. The updater module fails CLOSED if `gh` is absent (it
55-
// cannot verify attestations), so a host genuinely without `gh` needs an
56-
// explicit way to proceed. Default false: verification stays on in
57-
// production. Mirrors the --state-path pattern with an env fallback.
54+
// checksums.txt. Verification is performed in-process via sigstore-go (no
55+
// `gh` CLI or external tooling required) and fails CLOSED if provenance
56+
// cannot be established. This flag exists only for test/air-gapped
57+
// environments; leave it off in production. Default false: verification
58+
// stays on. Mirrors the --state-path pattern with an env fallback.
5859
skipAttestation := flag.Bool("skip-attestation", envBool("PILOT_UPDATER_SKIP_ATTESTATION"),
59-
"skip SLSA attestation verification (default off); use only on hosts without `gh` available")
60+
"skip SLSA attestation verification (default off); for test/air-gapped use only — production verifies in-process, no `gh` needed")
6061
flag.Parse()
6162

6263
if *showVersion {

0 commit comments

Comments
 (0)