-
-
Notifications
You must be signed in to change notification settings - Fork 0
chore(deps): Update GitHub Actions #78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,7 +30,7 @@ permissions: | |
| jobs: | ||
| ci: | ||
| name: CI Pipeline | ||
| uses: ByronWilliamsCPA/.github/.github/workflows/python-ci.yml@74c633acfdd5f707ab154fd59bd212c6df663dd6 # main | ||
| uses: ByronWilliamsCPA/.github/.github/workflows/python-ci.yml@7d12f5486ab5c856397ebaa4acd3c99ca385227c # main | ||
| with: | ||
| python-version: '3.12' | ||
| coverage-threshold: 80 | ||
|
|
@@ -55,14 +55,14 @@ jobs: | |
| working-directory: frontend | ||
| steps: | ||
| - name: Harden runner | ||
| uses: step-security/harden-runner@9af89fc71515a100421586dfdb3dc9c984fbf411 # v2.19.4 | ||
| uses: step-security/harden-runner@bf7454d06d71f1098171f2acdf0cd4708d7b5920 # v2.20.0 | ||
| with: | ||
| # egress-policy: block -- enabled 2026-05-23 (compliance audit). If a CI run fails on a network call, switch this single occurrence back to audit and capture the missing endpoint in the issue tracker. | ||
| egress-policy: audit # TODO: switch to block after 2026-06-30 (compliance audit deferral) | ||
| - name: Checkout | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 | ||
| uses: actions/checkout@d23441a48e516b6c34aea4fa41551a30e30af803 # v6.1.0 | ||
|
Comment on lines
62
to
+63
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Disable credential persistence on every modified checkout. These checkout steps still use the default credential persistence behavior. Add Suggested patch - name: Checkout repository
uses: actions/checkout@...
+ with:
+ persist-credentials: false
Based on learnings: modified checkout steps in touched workflows must explicitly disable credential persistence; as per path instructions, this is required workflow security hardening, and zizmor reports the same warning. 🧰 Tools🪛 zizmor (1.28.0)[warning] 62-63: credential persistence through GitHub Actions artifacts (artipacked): does not set persist-credentials: false (artipacked) 📍 Affects 7 files
🤖 Prompt for AI AgentsSources: Path instructions, Learnings, Linters/SAST tools |
||
| - name: Setup Node | ||
| uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0 | ||
| uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0 | ||
| with: | ||
| node-version: "22" | ||
| cache: "npm" | ||
|
|
@@ -92,7 +92,7 @@ jobs: | |
| if: always() | ||
| steps: | ||
| - name: Harden runner | ||
| uses: step-security/harden-runner@9af89fc71515a100421586dfdb3dc9c984fbf411 # v2.19.4 | ||
| uses: step-security/harden-runner@bf7454d06d71f1098171f2acdf0cd4708d7b5920 # v2.20.0 | ||
| with: | ||
| # egress-policy: block -- enabled 2026-05-23 (compliance audit). If a CI run fails on a network call, switch this single occurrence back to audit and capture the missing endpoint in the issue tracker. | ||
| egress-policy: audit # TODO: switch to block after 2026-06-30 (compliance audit deferral) | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔒 Security & Privacy | 🟠 Major | 🏗️ Heavy lift
🧩 Analysis chain
🏁 Script executed:
Repository: ByronWilliamsCPA/rag-processor
Length of output: 263
🏁 Script executed:
Repository: ByronWilliamsCPA/rag-processor
Length of output: 4206
Resolve the expired harden-runner egress deferrals.
These workflow steps remain in
egress-policy: auditeven though the inline comments require switching toblockafter June 30, 2026. Enable blocking after validating required endpoints, or renew the exception with an owner and a new expiry.Suggested patch
.github/workflows/pr-validation.yml#L55-L56: Expired dead-code egress deferral..github/workflows/pr-validation.yml#L98-L99: Expired link-check egress deferral..github/workflows/ci.yml#L60-L61: Expired Playwright/CI egress deferral.📍 Affects 2 files
.github/workflows/ci.yml#L60-L61(this comment).github/workflows/pr-validation.yml#L55-L56.github/workflows/pr-validation.yml#L98-L99🤖 Prompt for AI Agents
Sources: Coding guidelines, Path instructions