Skip to content

feat: Support attestation bundle URLs - #4550

Open
keeltrace wants to merge 1 commit into
google:masterfrom
keeltrace:keeltrace/attestation-bundle-url-4077
Open

keeltrace wants to merge 1 commit into
google:masterfrom
keeltrace:keeltrace/attestation-bundle-url-4077

Conversation

@keeltrace

Copy link
Copy Markdown
Contributor

GitHub REST API version 2026-03-10 returns artifact attestations using bundle_url instead of embedding bundle.

This keeps the existing Bundle field for older API/GHES responses, adds BundleURL, and updates the repository attestation test to cover both response shapes.

Part of #4077.

Testing: local Go execution was unavailable in the Keeltrace runner, so I did not claim local test results; repository CI is the executable verification for this submission.

AI-assisted contribution: ChatGPT helped inspect the migration issue, refresh the branch against current master, prepare the compatibility change, and review the submitted diff. I reviewed the scope and exact changes before submission.

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Sep 13, 2026
@gmlewis

gmlewis commented Sep 13, 2026

Copy link
Copy Markdown
Collaborator

Please run step 4 in CONTRIBUTING.md and push the changes to this PR and then we can proceed.

@codecov

codecov Bot commented Sep 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.57%. Comparing base (4340312) to head (b8f65fa).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4550      +/-   ##
==========================================
+ Coverage   98.53%   98.57%   +0.03%     
==========================================
  Files         195      197       +2     
  Lines       17851    18252     +401     
==========================================
+ Hits        17590    17991     +401     
  Misses        261      261              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

NeedsReview PR is awaiting a review before merging.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants