Feature: audit for Dependabot configs that miss local composite actions #1805
Replies: 2 comments
-
|
Hi @gcl-sekoia, thanks for opening this! TIL, I think that's pretty surprising behavior from Dependabot. In terms of scope/fit: I think this kind of check would be in scope for zizmor, but it's architecturally nontrivial for the reasons you mentioned: zizmor currently doesn't assume much (or anything) about the layout of files, and supports input modes where the "root" directory isn't obvious. So supporting this would be somewhat complicated. Another option here would be to raise this as a Dependabot bug: I think one could make a very strong argument that this behavior on their side is just plain wrong (and arguably insecure too), since |
Beta Was this translation helpful? Give feedback.
-
|
(Converting this one into an issue since I think it's a good feature request.) |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
When a
dependabot.ymlconfigures thegithub-actionsecosystem withdirectory: "/"(which is what most guides recommend), Dependabot updates action references in.github/workflows/but silently skips any local composite actions (e.g..github/actions/my-action/action.yml). Third-party action references inside those composite actions never get version update PRs.The fix is switching to
directorieswith explicit coverage:This seems like a good fit for a Dependabot audit since the misconfiguration is subtle and has supply chain implications (stale transitive action pins). zizmor already has both pieces of data needed β it discovers local composite actions and parses the
directory/directoriesfield in Dependabot configs.The main consideration is that this audit would only be meaningful when scanning a full repository (both actions and Dependabot config together), not individual files. Severity-wise it's probably informational or low since the impact is missed update PRs rather than a direct vulnerability.
Curious whether this is something that fits zizmor's scope, or if it's too far into "Dependabot linting" territory vs. supply chain security.
Beta Was this translation helpful? Give feedback.
All reactions