Skip to content

Commit 3671c2f

Browse files
committed
fix: replace localhost with ip address in case localhost is ipv6 addr??
Norbert had an issue with his docker container where the healthcheck was timing out/failing. He diagnosed it as a ipv6 address bound to localhost. Not sure if this is the right fix. Might be better to determine where localhost is bound (V4 or V6 address) but I don't have an environment I can test with.
1 parent 27281f8 commit 3671c2f

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

CHANGES.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,9 @@ Fixed:
3333
configuration. (John Rouillard)
3434
- fix bogus javascript emitted by user_src_input macro. (John
3535
Rouillard)
36-
36+
- replaced hostname localhost with 127.0.0.1 in docker healthcheck
37+
script. Found/patch by Norbert Schlemmer. (John Rouillard)
38+
3739
Features:
3840

3941
- add support for authorized changes. User can be prompted to enter

scripts/Docker/roundup_healthcheck

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
11
#! /bin/sh
22

3+
# change this if you are not using
4+
localhost=127.0.0.1
5+
36
# if there are multiple trackers, d=demo t=tracker ...
47
# returns last one for testing that server is up. Does not test
58
# each tracker.
69
tracker=$(ps -ef | sed -ne '/roundup-server/s/^.*\s\(\w*\)=.*$/\1/p')
7-
wget -q -O /dev/null --proxy off --no-verbose http://localhost:8080/"${tracker:-demo}"/
10+
wget -q -O /dev/null --proxy off --no-verbose http://$localhost:8080/"${tracker:-demo}"/

0 commit comments

Comments
 (0)