Skip to content

[LOW] Pin CLA action to an immutable commit - #2499

Open
OskarEichler wants to merge 1 commit into
Shopify:mainfrom
OskarEichler:codex/security-pin-cla-action
Open

[LOW] Pin CLA action to an immutable commit#2499
OskarEichler wants to merge 1 commit into
Shopify:mainfrom
OskarEichler:codex/security-pin-cla-action

Conversation

@OskarEichler

Copy link
Copy Markdown

Description

The CLA workflow currently references the mutable Shopify/shopify-cla-action@v1 tag while passing both the repository token and CLA_TOKEN to that action. If the tag is moved unexpectedly, the workflow would execute different code with those credentials.

This pins the action to the commit currently referenced by v1, while retaining the version comment for Dependabot and human readability. Runtime behavior is unchanged.

Severity: LOW. This is supply-chain hardening; no compromise or active exploitation is known or claimed.

Reviewers’ hat-rack 🎩

  • Confirm 9938f4b43524d1cfa7471ce9a803edf226697284 is the intended v1 revision.
  • Confirm the CLA workflow still receives the same inputs.

Verification:

  • Workflow YAML parses successfully.
  • The pinned SHA matches the current upstream refs/tags/v1 target.
  • The repository baseline passes: 14 Jest suites / 187 tests, type-check, lint, and build.
  • git diff --check passes.

No regression test was added because this is a one-line declarative action reference; the exact ref and YAML structure were checked directly.

Screenshots or videos (if needed)

Not applicable.

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