Skip to content

chore: ensure consistency in environment variable declarations in frontend dockerfile #1216

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore: ensure consistency in environment variable declarations in frontend dockerfile #1216

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions frontend/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,9 @@ WORKDIR /app

COPY package.json yarn.lock* package-lock.json* pnpm-lock.yaml* ./

COPY frontend/package*.json ./frontend/
COPY frontend/package*.json ./frontend/

COPY widget/package*.json ./widget/
COPY widget/package*.json ./widget/

RUN npm install

Expand All @@ -59,7 +59,7 @@ COPY ./widget ./widget


# used to by pass Next.js paching lock file
ENV NEXT_IGNORE_INCORRECT_LOCKFILE=true
ENV NEXT_IGNORE_INCORRECT_LOCKFILE=true
ENV NODE_ENV=development
ENV CHOKIDAR_USEPOLLING=true
ENV WATCHPACK_POLLING=true
Expand All @@ -72,11 +72,11 @@ CMD ["npm", "run", "dev", "--", "-p", "8080"]
FROM base AS production
WORKDIR /app

# used to by pass Next.js paching lock file
ENV NEXT_IGNORE_INCORRECT_LOCKFILE=true
# used to by pass Next.js paching lock file
ENV NEXT_IGNORE_INCORRECT_LOCKFILE=true
ENV NODE_ENV=production
# Uncomment the following line in case you want to disable telemetry during runtime.
ENV NEXT_TELEMETRY_DISABLED 1
ENV NEXT_TELEMETRY_DISABLED=1

RUN addgroup --system --gid 1001 nodejs
RUN adduser --system --uid 1001 nextjs
Expand All @@ -96,7 +96,7 @@ USER nextjs

EXPOSE 8080

ENV PORT 8080
ENV PORT=8080

# server.js is created by next build from the standalone output
# https://nextjs.org/docs/pages/api-reference/next-config-js/output
Expand Down