edgedb / edgedb-docker

Official Docker Image packaging for EdgeDB
81 stars 16 forks source link

Fix running as non-root #50

Closed elprans closed 2 years ago

elprans commented 2 years ago

Don't attempt to write outside of /tmp when the container isn't running as root.

fmoor commented 2 years ago

LGTM - but when do we need to run without root in Docker?

Heroku is the impetus for this change https://devcenter.heroku.com/articles/container-registry-and-runtime#run-the-image-as-a-non-root-user