Skip to content

Commit 13723bd

Browse files
Copiloteeholmes
andcommitted
Use chown with 755 permissions instead of 777 for better security
Co-authored-by: eeholmes <2545978+eeholmes@users.noreply.github.com>
1 parent aa9226e commit 13723bd

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ RUN wget -qO- https://raw.githubusercontent.com/retorquere/zotero-deb/master/ins
1616
# Copy the repo files into /tmp (environment/*.yml, install.R, etc)
1717
COPY . /tmp/
1818
# Fix permissions so NB_USER can write to /tmp/environment during package installs
19-
RUN chmod -R 777 /tmp/environment
19+
RUN chown -R ${NB_USER}:${NB_USER} /tmp/environment && chmod -R 755 /tmp/environment
2020

2121
# Update conda env sequentially to prevent hairy solve
2222
RUN /pyrocket_scripts/install-conda-packages.sh /tmp/environment/env-core1.yml || (echo "install env-core1 failed" && exit 1)

0 commit comments

Comments
 (0)