sourceboat / docker-laravel

A highly opinionated docker image which aims to be perfectly suited to run our Laravel applications.
MIT License
10 stars 2 forks source link

Idea: remove app user from dockerfile #25

Closed PKuebler closed 4 years ago

PKuebler commented 4 years ago

The standard user for web applications on Linux systems is "www-data". This user already exists. Therefore the user app is superfluous and is not used anywhere.

pehbehbeh commented 4 years ago

I merged #29 although the user home folder /home/app is used to store some scripts (see). We could move these scripts?

@PKuebler @hingew

hingew commented 4 years ago

We could move the scripts into /root/, @pehbehbeh what do you think ?

pehbehbeh commented 4 years ago

Sounds good to me but we need to change all run commands (also in readme).