[-] fix memory leak in Patroni checker, fixes #285

pashagolub · pashagolub · commit 164b218b7ccd · 2024-12-09T16:47:42.000+01:00
diff --git a/checker/patroni_leader_checker.go b/checker/patroni_leader_checker.go
@@ -34,6 +34,7 @@ func (c *PatroniLeaderChecker) GetChangeNotificationStream(ctx context.Context,
 				c.Logger.Sugar().Error("patroni REST API error:", err)
 				continue
 			}
+			r.Body.Close() //throw away the body
 			out <- strconv.Itoa(r.StatusCode) == c.TriggerValue
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,7 @@ func (c *PatroniLeaderChecker) GetChangeNotificationStream(ctx context.Context,`
`34`	`34`	`c.Logger.Sugar().Error("patroni REST API error:", err)`
`35`	`35`	`continue`
`36`	`36`	`}`
	`37`	`+ r.Body.Close() //throw away the body`
`37`	`38`	`out <- strconv.Itoa(r.StatusCode) == c.TriggerValue`
`38`	`39`	`}`
`39`	`40`	`}`