Static Analysis Report - 2026-03-09 #20167
Closed
Replies: 1 comment
-
|
This discussion was automatically closed because it expired on 2026-03-10T06:40:40.698Z.
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Analysis Summary
Today's scan reveals a major new finding: zizmor's
secrets-outside-envaudit now flags 3,541 instances across 165 of 166 workflows (Medium severity). This is a significant increase from yesterday's 4 total zizmor findings, almost certainly triggered by a zizmor update enabling this audit rule. All other findings (actionlint, poutine) remain stable.Findings by Tool
🚨 New Critical Finding: secrets-outside-env (Medium × 3,541)
NEW since 2026-03-09 — This audit rule was not triggered in any previous scan.
Zizmor's
secrets-outside-envrule detects when GitHub Actions secrets are referenced in workflow jobs without a dedicated GitHub Environment. Without environments, secrets have no additional protection layer (required reviewers, deployment branch rules, wait timers).Assessment: This spike is almost certainly caused by a zizmor tool update that enabled or introduced this audit rule. The finding itself represents real (if low-urgency) security hygiene — adding named GitHub Environments provides an extra access-control layer for secrets. Since all workflows are affected simultaneously with no code changes, this is not a regression in workflow code.
Clustered Findings by Tool and Type
Zizmor Security Findings
Poutine Supply Chain Findings (unchanged)
Actionlint Linting Issues (unchanged)
copilot-requestsactivatedoutput not definedFix Suggestion: secrets-outside-env
Issue: Secrets referenced without a dedicated GitHub Environment
Severity: Medium
Affected Workflows: 165 workflows (virtually all)
Prompt to Copilot Agent:
All Findings Details by Persistent Issue Type
Actionlint:
ace-editor— expression output undefined (PERSISTENT since 2026-03-06)ace-editor.lock.yml:525:9activatedis not defined in object type forneeds.activation.outputspost_ace_linkusesneeds.activation.outputs.activated == 'true'but theactivationjob does not declareactivatedin its outputs schemaactivatedto theactivationjob'soutputsdeclarationZizmor:
daily-copilot-token-report— artipacked (PERSISTENT since 2026-02-25)daily-copilot-token-report.lock.yml:304actions/checkoutpersists credentials in artifacts, enabling potential credential theft via artifact downloadpersist-credentials: falsetoactions/checkoutstepZizmor:
contribution-check— template-injection (PERSISTENT since 2026-02-25)contribution-check.lock.yml:317Poutine:
smoke-copilot-arm— pr_runs_on_self_hosted (PERSISTENT)smoke-copilot-arm.lock.yml:316ubuntu-24.04-arm— self-hosted runners in PR workflows can be exploited by malicious PRsPoutine: unverified_script_exec (PERSISTENT)
copilot-setup-steps.yml:17,daily-copilot-token-report.lock.yml:321curl | bashpattern usingraw.githubusercontent.com— executes unverified remote scriptsHistorical Trends
New Issues (2026-03-09):
secrets-outside-env(Medium): 3,541 instances across 165 workflows — NEW, likely from zizmor updateResolved Issues: None since last scan
Persistent Issues (unresolved): artipacked (13+ days), template-injection (13+ days), ace-editor expression error (4 days), all poutine findings (10+ days)
Recommendations
secrets-outside-envwas newly enabled in today's zizmor version. If this is a new audit rule, evaluate whether addingenvironment:declarations to all workflows is feasible or desired.activatedto theactivationjob's declared outputs inace-editor.md.persist-credentials: falseto theactions/checkoutstep.secrets-outside-env, consider a batch fix using the Copilot agent fix prompt above once the team decides on an environment naming strategy.Next Steps
secrets-outside-envat scaleproduction,default, workflow-specific)secrets-outside-envusing Copilot agent prompt aboveace-editorexpression output declaration (quick fix)artipackedindaily-copilot-token-reportReferences:
Beta Was this translation helpful? Give feedback.
All reactions