Add EXPOSE 80 in Dockerfile
tonito659 opened this issue · 3 comments
tonito659 commented
Hi ! I'm using Seafile behind jwilder/nginx-proxy. This container binds to the docker socket and scans running containers to facilitate reverse-proxying them.
But to automate it, it needs to know which ports are exposed by the container. Thoses ports are defined while building the image using EXPOSE (port number)
In addition this is good practice as it can help building on top of an image.
ohrensessel commented
As EXPOSE is given in the dockerfile of the pro version, I do not see any reason why it shouldn't be present in the normal version? came accross issue #28 and was wondering why it hasn't been merged...
tonito659 commented
Thanks a lot !