Skip to content

fix: Added .dockerignore and improved security#244

Open
Thunder-Blaze wants to merge 1 commit into
hoytech:masterfrom
Thunder-Blaze:feat/add-dockerignore
Open

fix: Added .dockerignore and improved security#244
Thunder-Blaze wants to merge 1 commit into
hoytech:masterfrom
Thunder-Blaze:feat/add-dockerignore

Conversation

@Thunder-Blaze

Copy link
Copy Markdown
Contributor

Description

  • There was no .dockerignore file present which cause COPY . . to copy all the local files into the image which in turn caused the build to failed
  • Improved security of the image by running under unprivileged users
  • Switched to apt-get (not necessary but better to use apt-get)

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.

1 participant