MirrorHub/conduit
1
0
Fork 0
mirror of https://gitlab.com/famedly/conduit.git synced 2024-11-19 23:02:04 +01:00
Code Issues Releases Activity

fix: Use default port for healthcheck as fallback

Browse source 
Conduit can start without a specific port being configured.
This adjusts the healthcheck script to tolerate that state.

Closes https://gitlab.com/famedly/conduit/-/issues/222
This commit is contained in:
Jonas Zohren 2022-01-28 22:19:19 +01:00
parent 296c68c0cd
commit 44f7a85077
No known key found for this signature in database
GPG key ID: FE3ED5D90A175463
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
docker/healthcheck.sh
View file

@ -1,10 +1,15 @@
#!/bin/sh #!/bin/sh
# If the port is not specified as env var, take it from the config file # If the port is not specified as env var, take it from the config file
if [ -z ${CONDUIT_PORT} ]; then if [ -z "${CONDUIT_PORT}" ]; then
CONDUIT_PORT=$(grep -m1 -o 'port\s=\s[0-9]*' conduit.toml | grep -m1 -o '[0-9]*') CONDUIT_PORT=$(grep -m1 -o 'port\s=\s[0-9]*' conduit.toml | grep -m1 -o '[0-9]*')
fi fi
# If the config file also does not contain a default port, just use the default one: 6167.
if [ -z "${CONDUIT_PORT}" ]; then
CONDUIT_PORT=6167
fi
# The actual health check. # The actual health check.
# We try to first get a response on HTTP and when that fails on HTTPS and when that fails, we exit with code 1. # We try to first get a response on HTTP and when that fails on HTTPS and when that fails, we exit with code 1.
# TODO: Change this to a single wget call. Do we have a config value that we can check for that? # TODO: Change this to a single wget call. Do we have a config value that we can check for that?