From 4252af29f4fc97125602cc8f3113c4804718c39a Mon Sep 17 00:00:00 2001 From: Jacob Kopczynski Date: Thu, 12 May 2022 07:52:53 -0700 Subject: [PATCH] Drop EXPOSE from Dockerfile (#302) There doesn't seem to be a good way to make this respect a param or envvar passed in, and it's not strictly needed if you're mapping ports. Having it be present & incorrect in some cases is worse than having it be absent in some cases where it would be correct. So just drop it entirely. Signed-off-by: Jacob Kopczynski --- Dockerfile | 3 --- 1 file changed, 3 deletions(-) diff --git a/Dockerfile b/Dockerfile index ddd0456d2..694acc536 100644 --- a/Dockerfile +++ b/Dockerfile @@ -63,8 +63,5 @@ COPY --from=BUILD_IMAGE $EXPLORER_APP_PATH/dist ./app/ COPY --from=BUILD_IMAGE $EXPLORER_APP_PATH/client/build ./client/build/ COPY --from=BUILD_IMAGE $EXPLORER_APP_PATH/node_modules ./node_modules/ -# expose default ports -EXPOSE 8080 - # run blockchain explorer main app CMD npm run app-start && tail -f /dev/null