Demo - [Guidelines] - Adding Dockerfile

[wvl94]

💡 PR Summary generated by FirstMate

Overview: Added a Dockerfile to containerize the application for easier deployment.

Changes:
New Dockerfile:

• Created /Dockerfile to define the application environment.
• Set base image to node:19-alpine for a lightweight container.
• Configured working directory to /usr/src/app.
• Installed git using Alpine package manager.
• Copied application files into the container.
• Exposed port 8080 for application access.
• Defined command to start the application with npm.

TLDR: A Dockerfile was added to facilitate application containerization; focus on the configuration and commands defined within.

_{^{Generated by FirstMate and automatically updated on every commit.}}

[firstmate]

PR Review

⚠️ It seems that you can still improve the quality of your PR:

    ❌ Docker & containerization: Optimize Dockerfiles with multi-stage builds and avoid running as root for security.

_{^{Generated by Firstmate to make sure you can focus on coding new features.}}

[firstmate]

Your Dockerfile should implement multi-stage builds to optimize image size and improve build performance. You can create a builder stage to install dependencies and build the application, then copy only the necessary files to a smaller runtime image. Here's an example:

FROM node:19-alpine AS builder
WORKDIR /usr/src/app
COPY package*.json ./
RUN npm install
COPY . .

FROM node:19-alpine
WORKDIR /usr/src/app
COPY --from=builder /usr/src/app .
EXPOSE 8080
CMD ["npm", "start", "--no-update-notifier"]

[firstmate]

The Dockerfile is currently running as the root user, which poses a security risk. It's better to create a non-root user and switch to it by adding the following lines to your Dockerfile:

RUN addgroup -S appgroup && adduser -S appuser -G appgroup
USER appuser