Skip to content

Commit 771ffdc

Browse files
committed
🔥(y-provider) remove npm in docker image
We use yarn and not npm, we remove npm because it has a dependencie with cross-spawn which has a CVE.
1 parent 82eba1e commit 771ffdc

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/frontend/servers/y-provider/Dockerfile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ COPY --from=y-provider-builder \
3131

3232
RUN NODE_ENV=production yarn install --frozen-lockfile
3333

34+
# Remove npm, contains CVE related to cross-spawn and we don't use it.
35+
RUN rm -rf /usr/local/bin/npm /usr/local/lib/node_modules/npm
36+
3437
# Un-privileged user running the application
3538
ARG DOCKER_USER
3639
USER ${DOCKER_USER}

0 commit comments

Comments
 (0)