admin/gluetun
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Maint: change default ping address to github.com to test DNS

Browse Source
This commit is contained in:
Quentin McGaw (desktop)
2021-09-11 22:27:32 +00:00
parent 6627cda96c
commit cc677bde93
5 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Dockerfile
View File

@@ -125,7 +125,7 @@ ENV VPNSP=pia \
LOG_LEVEL=info \
# Health
HEALTH_SERVER_ADDRESS=127.0.0.1:9999 \
HEALTH_ADDRESS_TO_PING=1.1.1.1 \
HEALTH_ADDRESS_TO_PING=github.com \
HEALTH_OPENVPN_DURATION_INITIAL=6s \
HEALTH_OPENVPN_DURATION_ADDITION=5s \
# DNS over TLS

2
internal/configuration/health.go
View File

@@ -52,7 +52,7 @@ func (settings *Health) read(r reader) (err error) {
return fmt.Errorf("environment variable HEALTH_SERVER_ADDRESS: %w", err)
}
settings.AddressToPing, err = r.env.Get("HEALTH_ADDRESS_TO_PING", params.Default("1.1.1.1"))
settings.AddressToPing, err = r.env.Get("HEALTH_ADDRESS_TO_PING", params.Default("github.com"))
if err != nil {
return fmt.Errorf("environment variable HEALTH_ADDRESS_TO_PING: %w", err)
}

4
internal/configuration/health_test.go
View File

@@ -49,7 +49,7 @@ func Test_Health_lines(t *testing.T) {
"filled settings": {
settings: Health{
ServerAddress: "address:9999",
AddressToPing: "1.1.1.1",
AddressToPing: "github.com",
VPN: HealthyWait{
Initial: time.Second,
Addition: time.Minute,
@@ -58,7 +58,7 @@ func Test_Health_lines(t *testing.T) {
lines: []string{
"|--Health:",
" |--Server address: address:9999",
" |--Address to ping: 1.1.1.1",
" |--Address to ping: github.com",
" |--VPN:",
" |--Initial duration: 1s",
" |--Addition duration: 1m0s",

4
internal/healthcheck/health_ping_test.go
View File

@@ -19,8 +19,8 @@ func Test_healthCheck_ping(t *testing.T) {
address string
err error
}{
"1.1.1.1": {
address: "1.1.1.1",
"github.com": {
address: "github.com",
},
"99.99.99.99": {
address: "99.99.99.99",

2
internal/healthcheck/health_test.go
View File

@@ -70,7 +70,7 @@ func Test_healthCheck(t *testing.T) {
t.Run("canceled real pinger", func(t *testing.T) {
t.Parallel()
pinger := newPinger("1.1.1.1")
pinger := newPinger("github.com")
canceledCtx, cancel := context.WithCancel(context.Background())
cancel()