Skip to content

ci: add ignore-scripts to Node package manager config (20260527-155221)#84

Merged
vallieres merged 2 commits into
mainfrom
sec-cli/ignore-scripts-fix-20260527-155221
Jun 5, 2026
Merged

ci: add ignore-scripts to Node package manager config (20260527-155221)#84
vallieres merged 2 commits into
mainfrom
sec-cli/ignore-scripts-fix-20260527-155221

Conversation

@securityeng-bot

Copy link
Copy Markdown
Contributor

This PR was generated by sec-cli github repositories ignore-scripts update.

What changed

Package managers detected: npm

Created config files:

  • .npmrc

Dockerfiles updated (COPY added before install step):

  • Dockerfile

Why

Running npm install (or yarn, pnpm, bun, npx) without restricting
postinstall scripts allows dependency lifecycle hooks to execute arbitrary code
during installation. Setting ignore-scripts=true (or the equivalent for your
package manager) mitigates this risk.

References:

@securityeng-bot
securityeng-bot Bot requested a review from a team as a code owner May 27, 2026 15:52
@vallieres

Copy link
Copy Markdown

@docker/registry please review and merge when you are available. 🙇🏻

@vallieres vallieres left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving security hardening PR: adds ignore-scripts=true to npm config to prevent supply-chain attacks via postinstall hooks. Change is minimal and correct.

@vallieres
vallieres merged commit ad806e2 into main Jun 5, 2026
5 checks passed
@vallieres
vallieres deleted the sec-cli/ignore-scripts-fix-20260527-155221 branch June 5, 2026 13:23
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