Context cancellation needs no stack trace.

mdwhatcott · mdwhatcott · commit 85308b4e22f2 · 2025-09-17T16:19:33.000-06:00
diff --git a/sqldb.go b/sqldb.go
@@ -59,9 +59,8 @@ func NormalizeErr(err error) error {
 	if err == nil {
 		return nil
 	}
-	err = fmt.Errorf("%w\nStack Trace:\n%s", err, string(debug.Stack()))
 	if strings.Contains(err.Error(), "operation was canceled") {
 		return fmt.Errorf("%w: %w", context.Canceled, err)
 	}
-	return err
+	return fmt.Errorf("%w\nStack Trace:\n%s", err, string(debug.Stack()))
 }

Original file line number	Diff line number	Diff line change
`@@ -59,9 +59,8 @@ func NormalizeErr(err error) error {`
`59`	`59`	`if err == nil {`
`60`	`60`	`return nil`
`61`	`61`	`}`
`62`		`- err = fmt.Errorf("%w\nStack Trace:\n%s", err, string(debug.Stack()))`
`63`	`62`	`if strings.Contains(err.Error(), "operation was canceled") {`
`64`	`63`	`return fmt.Errorf("%w: %w", context.Canceled, err)`
`65`	`64`	`}`
`66`		`- return err`
	`65`	`+ return fmt.Errorf("%w\nStack Trace:\n%s", err, string(debug.Stack()))`
`67`	`66`	`}`