feature/firstmate-docker-align-06398f1c #31
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dockerfile Optimization
I have optimized the Dockerfile to align with company best practices and improve security and efficiency. Here are the changes made:
➡️ Updated Node.js version: Upgraded from
node:19-alpine
tonode:20-alpine
for better performance and security updates.➡️ Multi-stage build: Implemented a multi-stage build to reduce the final image size and improve build efficiency.
➡️ Non-root user: Added a non-root user to enhance security. Running containers as root is a security risk.
➡️ Layer optimization: Reordered instructions to optimize Docker layer caching, which speeds up the build process.
These changes ensure that the Dockerfile adheres to the company's guidelines, enhancing security and efficiency.