clamav-rest-api icon indicating copy to clipboard operation
clamav-rest-api copied to clipboard

Update Dockerfile to expose 8080

Open cbarange opened this issue 1 year ago • 1 comments

Hello I saw that 8080 wasn't expose in the dockerfile. So just fix it

cbarange avatar Sep 11 '23 21:09 cbarange

Hmmm. The port on which CRA will listen to is defined by ENV variable: APP_PORT and the default is (as I remember) 3000. I'm not sure it will be valid to include EXPOSE clause in the Dockerfile as the listen/expose port can be changed by the one who start the container. Have a look also here: https://docs.docker.com/engine/reference/builder/#expose - it looks like EXPOSE is a kind of "documentation"

benzino77 avatar Sep 13 '23 10:09 benzino77