Skip to content

fix(security): add .npmrc with ignore-scripts=true#1083

Open
yogeshchoudhary147 wants to merge 1 commit intomainfrom
ci_ignore_scripts
Open

fix(security): add .npmrc with ignore-scripts=true#1083
yogeshchoudhary147 wants to merge 1 commit intomainfrom
ci_ignore_scripts

Conversation

@yogeshchoudhary147
Copy link
Copy Markdown
Contributor

@yogeshchoudhary147 yogeshchoudhary147 commented Apr 10, 2026

This prevents malicious postinstall scripts from running during npm install, both in CI and locally.

Changes

  • Added .npmrc with ignore-scripts=true
  • Removed .npmrc from .gitignore (auth tokens belong in ~/.npmrc, not project-level)

This is safe because the SDK and its dependencies are pure JavaScript with no native compilation or required lifecycle scripts.

@yogeshchoudhary147 yogeshchoudhary147 requested a review from a team as a code owner April 10, 2026 05:25
@yogeshchoudhary147 yogeshchoudhary147 changed the title fix(security): add --ignore-scripts to CI npm installs fix(security): add .npmrc with ignore-scripts=true Apr 10, 2026
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