Skip to content

Commit 074abac

Browse files
committed
chore: remove HEALTHCHECK from production Dockerfile
I think this should not be set in the image, especially because it has to rely on some assumptions. Instead, people wanting a health check for their container, should add it themselves. Mc-Auth doesn't event have a /status endpoint or anything right now.
1 parent db0d379 commit 074abac

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

Dockerfile

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,6 @@ CMD ["node", "--enable-source-maps", "dist/index.js"]
4949
##
5050
FROM base AS prod
5151

52-
# TODO: This heavily relies on hostname being set and the default port 8080 being used
53-
HEALTHCHECK --interval=1m --timeout=30s --retries=3 \
54-
CMD wget --spider $(hostname):8080
55-
5652
ENV NODE_ENV=production
5753
RUN npm clean-install && \
5854
npm cache clean --force && \

0 commit comments

Comments
 (0)