Skip to content

Commit

Permalink
fix: shorter waiting for ping (#137)
Browse files Browse the repository at this point in the history
  • Loading branch information
jan-hajek authored Mar 22, 2024
1 parent 99f76e4 commit 6ce7b12
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,10 +84,10 @@ func rootCmd() *cmdBuilder.Cmd {
}
}

ctx, cancel := context.WithTimeout(ctx, time.Second*5)
pingCtx, cancel := context.WithTimeout(ctx, time.Second)
defer cancel()

if err := nettools.Ping(ctx, vpnCheckAddress); err != nil {
if err := nettools.Ping(pingCtx, vpnCheckAddress); err != nil {
body.AddStringsRow(i18n.T(i18n.StatusInfoVpnStatus), i18n.T(i18n.VpnCheckingConnectionIsNotActive))
} else {
body.AddStringsRow(i18n.T(i18n.StatusInfoVpnStatus), i18n.T(i18n.VpnCheckingConnectionIsActive))
Expand Down
4 changes: 2 additions & 2 deletions src/cmd/vpnUp.go
Original file line number Diff line number Diff line change
Expand Up @@ -184,10 +184,10 @@ func isVpnConnect(ctx context.Context, uxBlocks uxBlock.UxBlocks) bool {
p := []uxHelpers.Process{
{
F: func(ctx context.Context) error {
ctx, cancel := context.WithTimeout(ctx, time.Second*5)
pingCtx, cancel := context.WithTimeout(ctx, time.Second)
defer cancel()

return nettools.Ping(ctx, vpnCheckAddress)
return nettools.Ping(pingCtx, vpnCheckAddress)
},
RunningMessage: i18n.T(i18n.VpnCheckingConnection),
ErrorMessageMessage: i18n.T(i18n.VpnCheckingConnectionIsNotActive),
Expand Down

0 comments on commit 6ce7b12

Please sign in to comment.