Node user should be supplied

smebberson / docker-alpine

Docker containers running Alpine Linux and s6 for process management. Solid, reliable containers.

MIT License

596 stars 186 forks source link

Node user should be supplied #12

Open matthewvalimaki opened 8 years ago

matthewvalimaki commented 8 years ago

alpine-consul-nginx-nodejs and alpine-nginx-nodejs should provide nodejs user. user-consul-nginx-nodejs and user-nginx-nodejs then both use this user in s6 run scripts. Right now node server is running as root.

smebberson commented 8 years ago

@matthewvalimaki, true! Not a good scenario to have them running as root. I'll get this added.

matthewvalimaki commented 8 years ago

See pull request #33 for resolution.