Merge pull request #3404 from itsmatinx/main

Release ticker resources
2021-05-05 16:40:13 +08:00 · 2021-05-05 16:40:13 +08:00 · d0deff9cd6
commit d0deff9cd6
parent a9d8a05d08 835651e513
1 changed files with 2 additions and 0 deletions
--- a/health/health.go
+++ b/health/health.go
@ -135,6 +135,7 @@ func PeriodicChecker(check Checker, period time.Duration) Checker {
 	u := NewStatusUpdater()
 	go func() {
 		t := time.NewTicker(period)
+		defer t.Stop()
 		for {
 			<-t.C
 			u.Update(check.Check())
@ -150,6 +151,7 @@ func PeriodicThresholdChecker(check Checker, period time.Duration, threshold int
 	tu := NewThresholdStatusUpdater(threshold)
 	go func() {
 		t := time.NewTicker(period)
+		defer t.Stop()
 		for {
 			<-t.C
 			tu.Update(check.Check())