Skip to content

Fix: Pin metadata validator action to audited commit - #511

Open
localtrade1 wants to merge 1 commit into
symbioticfi:mainfrom
localtrade1:localtrade1-patch-1
Open

Fix: Pin metadata validator action to audited commit#511
localtrade1 wants to merge 1 commit into
symbioticfi:mainfrom
localtrade1:localtrade1-patch-1

Conversation

@localtrade1

Copy link
Copy Markdown

Description

This PR addresses a supply-chain security risk identified during the workspace-wide audit by eliminating a mutable GitHub Action reference in the CI pipeline[cite: 34].

Key Changes

  • Mutable Action Reference (.github/workflows/validate-pr.yml): The metadata validator action was previously pointing to the mutable main branch[cite: 34]. To ensure execution consistency and prevent upstream supply-chain attacks, the action reference is now strictly pinned to the audited commit SHA (a21a6f01a2b6c5fda769e0c7c0b55d79172568f6)[cite: 34].

### Description
This PR addresses a supply-chain security risk identified during the workspace-wide audit by eliminating a mutable GitHub Action reference in the CI pipeline[cite: 34].

### Key Changes
* **Mutable Action Reference (`.github/workflows/validate-pr.yml`):** The metadata validator action was previously pointing to the mutable `main` branch[cite: 34]. To ensure execution consistency and prevent upstream supply-chain attacks, the action reference is now strictly pinned to the audited commit SHA (`a21a6f01a2b6c5fda769e0c7c0b55d79172568f6`)[cite: 34].
@github-actions

Copy link
Copy Markdown

We detected changes in the pull request that are not allowed. Please, follow the contribution guidelines.

Not allowed files:

  • .github/workflows/validate-pr.yml

@localtrade1

Copy link
Copy Markdown
Author

Hi team, this PR specifically aims to fix the GitHub Actions workflow by pinning the validator action to an audited commit, as reflected in the title.

However, the validate-pr check is automatically failing because external contributors are restricted from modifying files in the .github/workflows/ directory. Could a maintainer please review the changes and override the check or merge this manually?

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.

1 participant