-
Notifications
You must be signed in to change notification settings - Fork 13
🚨 Update github actions (main) (major) #528
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
base: main
Are you sure you want to change the base?
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 |
|---|---|---|
|
|
@@ -32,17 +32,17 @@ jobs: | |
| HUGO_VERSION: 0.111.3 | ||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 | ||
| uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 | ||
|
|
||
| - name: Setup NodeJS environment | ||
| uses: actions/setup-node@395ad3262231945c25e8478fd5baf05154b1d79f # v6.1.0 | ||
| uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0 | ||
| with: | ||
| node-version-file: 'antora/package.json' | ||
| cache: 'npm' | ||
| cache-dependency-path: 'antora/package-lock.json' | ||
|
|
||
| - name: Setup Go environment | ||
| uses: actions/setup-go@4dc6199c7b1a012772edbd06daecab0f50c9053c # v6.1.0 | ||
| uses: actions/setup-go@b7ad1dad31e06c5925ef5d2fc7ad053ef454303e # v7.0.0 | ||
| with: | ||
| go-version-file: website/go.mod | ||
| cache: true | ||
|
|
@@ -71,7 +71,7 @@ jobs: | |
| --baseURL "${{ steps.pages.outputs.base_url }}/" | ||
|
|
||
| - name: Upload artifact | ||
| uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v6.0.0 | ||
| uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1 | ||
| with: | ||
| path: public | ||
| name: website | ||
|
|
@@ -85,7 +85,7 @@ jobs: | |
| mkdir pull_request | ||
|
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. [low] pin-integrity The SHA-to-tag mappings in this PR are generated by Renovate bot, whose core function is SHA-to-tag resolution. While independent verification of pinned SHAs is always good practice, the risk is low given the trusted provenance. A human reviewer may optionally verify each SHA maps to the stated tag. Suggested fix: For each action, run |
||
| echo ${{ github.event.pull_request.number }} > ./pull_request/number | ||
| - name: Upload pull request data | ||
| uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v6.0.0 | ||
| uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1 | ||
| with: | ||
| path: . | ||
| name: pull-request-data | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,7 +35,7 @@ jobs: | |
| if: github.event.workflow_run.event == 'pull_request' && github.event.workflow_run.conclusion == 'success' | ||
| steps: | ||
| - name: Download pull request artifact | ||
| uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0 | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0 | ||
| with: | ||
| script: | | ||
| const artifacts = await github.rest.actions.listWorkflowRunArtifacts({ | ||
|
|
@@ -67,7 +67,7 @@ jobs: | |
| echo "PR_NUMBER=${PR_NUMBER}" >> $GITHUB_OUTPUT | ||
|
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. [low] GHA workflow command injection Pre-existing: ${{ steps.data.outputs.PR_NUMBER }} is interpolated directly into JavaScript code via issue_number: Number(...). The value is sanitized via ${PR_NUMBER//[^0-9]/} (digits-only), limiting the injection surface. Not introduced by this PR. Suggested fix: For defense-in-depth, consider passing PR_NUMBER via an environment variable instead of direct expression interpolation. |
||
| echo "PR_URL=https://github.com/${GITHUB_REPOSITORY}/pull/${PR_NUMBER}" >> $GITHUB_OUTPUT | ||
| - name: Checkout | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 | ||
| uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 | ||
| with: | ||
| path: code | ||
| - name: Preview | ||
|
|
@@ -80,7 +80,7 @@ jobs: | |
| npm --prefix code exec -- wrangler pages deploy public --project-name "$CLOUDFLARE_PROJECT_NAME" --branch pr-${{ steps.data.outputs.PR_NUMBER }} | tee out.txt | ||
| grep 'Deployment complete! Take a peek over at ' out.txt | sed -e 's/.*over at /PREVIEW_URL=/' >> $GITHUB_OUTPUT | ||
| - name: Add comment | ||
| uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0 | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0 | ||
| with: | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| script: | | ||
|
|
||
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.
[low] Supply chain - action pin verification
All 8 action references are bumped to new major versions with full SHA pins and version comments. Verify that SHA pins match the claimed version tags against the official action repositories.