-
-
Save thom-nic/5e561e800cbc0f71555c to your computer and use it in GitHub Desktop.
### | |
# Node.js app Docker file | |
# | |
# Some basic build instructions: | |
# ``` | |
# # you should delete node_modules b/c you don't want that copied during 'ADD' | |
# docker build -t thom-nic/node-bootstrap . | |
# # run a shell in the container to inspect the environment (as root): | |
# docker run --rm -itu root thom-nic/node-bootstrap /bin/bash | |
# ``` | |
### | |
FROM dockerfile/nodejs | |
MAINTAINER Thom Nichols "[email protected]" | |
RUN useradd -ms /bin/bash node | |
# copy the nice dotfiles that dockerfile/ubuntu gives us: | |
RUN cd && cp -R .bash_profile .bashrc .gitconfig .profile scripts /home/node | |
ADD . /home/node/app | |
RUN chown -R node:node /home/node | |
USER node | |
ENV HOME /home/node | |
WORKDIR /home/node/app | |
#ENV NODE_ENV production | |
RUN npm install | |
EXPOSE 8888 | |
CMD ["npm", "start"] |
@thom-nic First off, thank you for the gist. I'm trying to accomplish something similar and stumbled across your SO post, and I wound up here.
It seems to me that one wouldn't want the files to be owned by the user that the application runs under (node
in this case). As a hypothetical, let's say that part of the node application allows the user to upload files. Furthermore, assume that the upload code fails to properly sanitize the upload path, thereby allowing users to control the destination. If node
owns the files, then a malicious user exploiting the upload vulnerability would be able to effectively overwrite application code. This scenario is a common attack vector.
If you consider webservers like apache and nginx, for the very reason outlined above you wouldn't want the application files to be owned by the same user as the webserver runs under, right? (e.g. granting ownership of php files to www-user
is a security issue.) In the Dockerfile, it therefore seems like root
should own the files, and the node
user should run the application.
Thoughts?
Note that lines 22 and 23 mean the
npm install
call is run as thenode
user. I have bower installed as a local dependency (not-g
) and use npm's postinstall hook to install bower assets:relevant bits of my
package.json
This means I don't need extra
npm install -g
lines in the dockerfile.