Skip to content

Fix: authenticate release verification and emit valid checksums - #138

Merged
0xLeif merged 3 commits into
mainfrom
leif/archive-release-auth-specsync
Jul 29, 2026
Merged

0xLeif merged 3 commits into
mainfrom
leif/archive-release-auth-specsync

Conversation

@0xLeif

@0xLeif 0xLeif commented Jul 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • reuse checkout-managed credentials for the post-publication tag fetch
  • remove the duplicate hand-built Authorization header
  • generate checksum sidecars without stripping the digest
  • verify every generated sidecar before release publication
  • strengthen regression coverage for authentication and SHA-256 content

Root causes

The final verification step retained a command-scoped Authorization header after the other release fetches moved to checkout-managed credentials. Separately, checksum generation passed ./asset through a greedy sed expression, which stripped the digest and left only the filename. The prior negative shell assertion used ! grep under set -e, which did not terminate the script when the forbidden pattern was present.

Test Plan

  • fledge run release-provenance-test
  • fledge run release-distribution-test
  • fledge lanes run verify (11 steps, 266 tests)

@cursor

cursor Bot commented Jul 29, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@0xLeif 0xLeif changed the title Fix: authenticate the post-release tag verification Fix: authenticate release verification and emit valid checksums Jul 29, 2026
@0xLeif
0xLeif merged commit 1c27f4f into main Jul 29, 2026
5 checks passed
@0xLeif
0xLeif deleted the leif/archive-release-auth-specsync branch July 29, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant