Skip to content

Commit bcebe0c

Browse files
authored
Remediate insecure npm usage in Docker image (#8786)
* Remediate insecure npm usage in Docker image * No -am
1 parent 683c68e commit bcebe0c

File tree

5 files changed

+8167
-4
lines changed

5 files changed

+8167
-4
lines changed

scripts/publish.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,10 +92,14 @@ echo "Publishing to npm..."
9292
npx [email protected] --before-script ./scripts/clean-shrinkwrap.sh
9393
echo "Published to npm."
9494

95+
echo "Updating package-lock.json for Docker image..."
96+
npm --prefix ./scripts/publish/firebase-docker-image install
97+
echo "Updated package-lock.json for Docker image."
98+
9599
echo "Cleaning up release notes..."
96100
rm CHANGELOG.md
97101
touch CHANGELOG.md
98-
git commit -m "[firebase-release] Removed change log and reset repo after ${NEW_VERSION} release" CHANGELOG.md
102+
git commit -m "[firebase-release] Removed change log and reset repo after ${NEW_VERSION} release" CHANGELOG.md scripts/publish/firebase-docker-image/package-lock.json
99103
echo "Cleaned up release notes."
100104

101105
echo "Pushing to GitHub..."

scripts/publish/firebase-docker-image/Dockerfile

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,20 @@ FROM node:lts-alpine AS app-env
33
# Install Python and Java and pre-cache emulator dependencies.
44
RUN apk add --no-cache python3 py3-pip openjdk11-jre bash && \
55
apk update && \
6-
apk upgrade && \
7-
npm install -g firebase-tools && \
8-
firebase setup:emulators:database && \
6+
apk upgrade
7+
8+
9+
RUN mkdir -p /usr/local/node_packages/
10+
COPY package.json /usr/local/node_packages/
11+
COPY package-lock.json /usr/local/node_packages/
12+
13+
WORKDIR /usr/local/node_packages/
14+
RUN npm install
15+
ENV PATH="/usr/local/node_packages/node_modules/.bin:${PATH}"
16+
17+
WORKDIR /
18+
19+
RUN firebase setup:emulators:database && \
920
firebase setup:emulators:firestore && \
1021
firebase setup:emulators:pubsub && \
1122
firebase setup:emulators:storage && \

0 commit comments

Comments
 (0)