Skip to content

allowlist: add carabiner-dev install/download-and-verify transitive dep#895

Open
potiuk wants to merge 1 commit into
mainfrom
allowlist-carabiner-download-and-verify-v117
Open

allowlist: add carabiner-dev install/download-and-verify transitive dep#895
potiuk wants to merge 1 commit into
mainfrom
allowlist-carabiner-download-and-verify-v117

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented May 28, 2026

Summary

  • Add carabiner-dev/actions/install/download-and-verify@v1.1.7
    (2a11d59a135c5e291f305f249a92ad7903e3ee0f) to the allowlist.
    Same monorepo commit already approved for install/ampel and
    install/bnd via allowlist: add carabiner-dev install/{ampel,bnd} transitive deps #831download-and-verify is a sibling
    subpath under the same SHA, reachable transitively from
    ampel/verify@v1.2.0.
  • Fixes the recurring check-for-transitive-failures failures
    (e.g. run 26602787860):
    GitHub Actions rejects the transitive download-and-verify@2a11d59a
    ref because it isn't on the org allowlist.

Test plan

  • After merge, update.yml regenerates approved_patterns.yml
    + the composite from actions.yml.
  • ASF Infra's allowlist sync picks up the new SHA; the next
    hourly check-for-transitive-failures run passes.

…itive

carabiner-dev/actions/ampel/verify@v1.2.0 (already approved) calls
install/download-and-verify at v1.1.7 SHA 2a11d59a — the same
monorepo commit already approved for install/{ampel,bnd} via #831.
GitHub Actions validates every transitive `uses:` against the org
allowlist, so this subpath needs to be approved too — without it,
check-for-transitive-failures fails every hour with "action ... is
not allowed".

Generated-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@potiuk potiuk requested review from dfoulks1 and ppkarwasz as code owners May 28, 2026 21:20
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented May 29, 2026

Ping @raboof @ppkarwasz @dfoulks1 — small allowlist add (same monorepo pattern as #831). Without this, the hourly check-for-transitive-failures workflow is red every cycle today (example). Would appreciate a quick approve so the next tick can self-heal.

Copy link
Copy Markdown
Contributor

@dfoulks1 dfoulks1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this LGTM

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.

2 participants