Skip to content

Fixed bad permissions scheme in dockerfile#189

Merged
sfreeman422 merged 4 commits intomasterfrom
fix/image-read-write
Mar 22, 2026
Merged

Fixed bad permissions scheme in dockerfile#189
sfreeman422 merged 4 commits intomasterfrom
fix/image-read-write

Conversation

@sfreeman422
Copy link
Collaborator

@sfreeman422 sfreeman422 commented Mar 22, 2026

This pull request makes a minor update to the Docker build process for the backend package, specifically improving the security of the images directory.

  • Dockerfile improvement:
    • In packages/backend/Dockerfile, after creating the /usr/src/app/images directory during the build, the directory's permissions are set to 700 to restrict access to the owner only.

Copilot AI review requested due to automatic review settings March 22, 2026 15:29
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens filesystem permissions for the backend runtime “images” directory created during the Docker build, aligning it with the intent that only the container’s app user should access it.

Changes:

  • Create /usr/src/app/images during the build stage and set it to chmod 700.
  • Preserve ownership for the runtime image via existing COPY --chown=65532:65532 ... ./images.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sfreeman422 sfreeman422 merged commit 4aef26e into master Mar 22, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants