Wrap debug.Stack() in string() (#4844)

bks71 · web-flow · commit b6ba82bb8215 · 2023-05-25T11:02:43.000-04:00
This is being flagged by Snyk as a security issue. In other places in the code, debug.Stack() is wrapped in string(). Just adding that here.
diff --git a/private/model/cli/gen-api/main.go b/private/model/cli/gen-api/main.go
@@ -162,7 +162,7 @@ func writeServiceFiles(g *generateInfo, pkgDir string) {
 	defer func() {
 		if r := recover(); r != nil {
 			fmt.Fprintf(os.Stderr, "Error generating %s\n%s\n%s\n",
-				pkgDir, r, debug.Stack())
+				    pkgDir, r, string(debug.Stack()))
 			os.Exit(1)
 		}
 	}()

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ func writeServiceFiles(g *generateInfo, pkgDir string) {`
`162`	`162`	`defer func() {`
`163`	`163`	`if r := recover(); r != nil {`
`164`	`164`	`fmt.Fprintf(os.Stderr, "Error generating %s\n%s\n%s\n",`
`165`		`- pkgDir, r, debug.Stack())`
	`165`	`+ pkgDir, r, string(debug.Stack()))`
`166`	`166`	`os.Exit(1)`
`167`	`167`	`}`
`168`	`168`	`}()`