panic: non-positive interval for NewTicker about mole HOT 8 CLOSED

On Tue, Sep 10, 2019 at 6:08 PM James Mills ***@***.***> wrote: I'm sort of at a loss here :/ $ sift KeepAliveInterval cli/cli.go:34: KeepAliveInterval time.Duration cli/cli.go:62: f.DurationVar(&c.KeepAliveInterval, "keep-alive-interval", 10*time.Second, "(optional) time interval for keep alive packets to be sent") tunnel/tunnel.go:125: // KeepAliveInterval is the time period used to send keep alive packets to tunnel/tunnel.go:127: KeepAliveInterval time.Duration tunnel/tunnel.go:300: ticker := time.NewTicker(t.KeepAliveInterval) tunnel/tunnel_test.go:352: tun.KeepAliveInterval = 10 * time.Second cmd/mole/main.go:276: log.Infof("t.KeepAliveInterval: #%v", app.KeepAliveInterval) cmd/mole/main.go:278: t.KeepAliveInterval = app.KeepAliveInterval Inspecting the code and debugging it a little bit shows that we are missing t.KeepAliveInterval from app.KeepAliveInterface which *should* be set from the f.DurationVar(&c.KeepAliveInterval on L62 of cli/cli.go — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#88?email_source=notifications&email_token=AAABEKR6SY2BJMWH6JNPJWDQJBAKNA5CNFSM4IVN46LKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6M52FA#issuecomment-530177300>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAABEKROODGWRHEFFDPJJADQJBAKNANCNFSM4IVN46LA> .