search

kekru / docker-remote-api-tls

Docker Image that forwards to the Docker API Socket and requires TLS Client authentication
https://hub.docker.com/r/kekru/docker-remote-api-tls
MIT License
76 stars 25 forks source link

updates nginx config to run workers as root to avoid changing permiss… #1

Closed wershlak closed 6 years ago

wershlak commented 7 years ago

updates nginx config to run workers as root to avoid changing permissions on docker.sock. Since /var/run/docker.sock isn't permanent (daemon restart will undo the chmod). Chmoding it to 0666 also adds vulnerability to the host this seemed like a better/safer solution.

kekru commented 6 years ago

Hi @wershlak oh, I'm so sorry, I totally forgot this PullRequest. It's already here for over a year. Sorry.

I just came back to the PullRequests, because of the new #2. It does nearly the same. Lets start the discussion there

kekru commented 6 years ago

this is now fixed in #2