Skip to content

chore: replace npm i with npm ci --ignore-scripts#526

Open
rensvmoorsel wants to merge 1 commit intomasterfrom
chore/replace-npm-i-with-npm-ci
Open

chore: replace npm i with npm ci --ignore-scripts#526
rensvmoorsel wants to merge 1 commit intomasterfrom
chore/replace-npm-i-with-npm-ci

Conversation

@rensvmoorsel
Copy link
Copy Markdown

No description provided.

@SilasPeters
Copy link
Copy Markdown
Member

Other places might also download node packages, like the systemd service for backup-to-s3

@SilasPeters
Copy link
Copy Markdown
Member

SilasPeters commented Sep 30, 2025

What needs to be done before we merge:

  • Replace all npm install with npm ci --no-scripts
  • Secure all other places where node is used (npx, npm exec, others?)
  • Secure dockerfiles for images used on sadserver (thus possibly requiring changes in other repositories)

@SilasPeters
Copy link
Copy Markdown
Member

SilasPeters commented Apr 1, 2026

Additions: fancy-index build script, and the radio rebuild handler

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.

3 participants