Skip to content

Commit b93f8d4

Browse files
authored
Added check and fix for global --install.ignore-scripts yarn config (#1451)
ref https://linear.app/ghost/issue/PRO-1540/ - there have been multiple recent npm incidents with compromised packages using pre/post-install scripts to run malicious scripts - we want to default to not running these scripts as a security precaution, this matches behaviour of pnpm which is touted as a modern, more secure, npm package manager
1 parent acc6097 commit b93f8d4

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ WORKDIR /opt/activitypub
88
COPY package.json .
99
COPY yarn.lock .
1010

11-
RUN yarn && \
11+
RUN yarn --ignore-scripts && \
1212
yarn cache clean
1313

1414
COPY tsconfig.json .

jobs/populate-explore-json/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ WORKDIR /app
66
COPY package.json yarn.lock ./
77

88
# Install dependencies
9-
RUN yarn install --frozen-lockfile --production
9+
RUN yarn install --ignore-scripts --frozen-lockfile --production
1010

1111
# Copy source
1212
COPY index.ts ./

0 commit comments

Comments
 (0)