Merge pull request #9669 from Kisesy/update-gin

msmith-techempower · web-flow · commit 0c85506f2997 · 2025-03-14T15:34:13.000-07:00
[go/gin] Ensure rows are closed
diff --git a/frameworks/Go/gin/gin-src/hello.go b/frameworks/Go/gin/gin-src/hello.go
@@ -99,6 +99,7 @@ func fortunes(c *gin.Context) {
 		c.AbortWithError(500, err)
 		return
 	}
+	defer rows.Close() // Ensure rows are closed
 
 	fortunes := make(Fortunes, 0)
 	for rows.Next() { //Fetch rows
diff --git a/frameworks/Go/gin/gin-std/main.go b/frameworks/Go/gin/gin-std/main.go
@@ -99,6 +99,7 @@ func fortunes(c *gin.Context) {
 		c.AbortWithError(500, err)
 		return
 	}
+	defer rows.Close() // Ensure rows are closed
 
 	fortunes := make(Fortunes, 0)
 	for rows.Next() { //Fetch rows

Original file line number	Diff line number	Diff line change
`@@ -99,6 +99,7 @@ func fortunes(c *gin.Context) {`
`99`	`99`	`c.AbortWithError(500, err)`
`100`	`100`	`return`
`101`	`101`	`}`
	`102`	`+ defer rows.Close() // Ensure rows are closed`
`102`	`103`
`103`	`104`	`fortunes := make(Fortunes, 0)`
`104`	`105`	`for rows.Next() { //Fetch rows`