Merge pull request #40 from fly-apps/fix-connection-leak

davissp14 · web-flow · commit 2e7cbbb2e6ce · 2023-01-19T08:54:01.000-06:00
Make sure connections get closed
diff --git a/cmd/standby_cleaner/main.go b/cmd/standby_cleaner/main.go
@@ -32,6 +32,7 @@ func main() {
 		fmt.Printf("failed to open local connection: %s\n", err)
 		os.Exit(1)
 	}
+	defer conn.Close(ctx)
 
 	internal, err := flypg.ReadFromFile("/data/flypg.internal.conf")
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,7 @@ func main() {`
`32`	`32`	`fmt.Printf("failed to open local connection: %s\n", err)`
`33`	`33`	`os.Exit(1)`
`34`	`34`	`}`
	`35`	`+ defer conn.Close(ctx)`
`35`	`36`
`36`	`37`	`internal, err := flypg.ReadFromFile("/data/flypg.internal.conf")`
`37`	`38`	`if err != nil {`