[telemetry] Don't log user errors

mikeland73 · mikeland73 · commit f68fafa44904 · 2024-09-26T22:20:04.000-07:00
diff --git a/internal/boxcli/midcobra/telemetry.go b/internal/boxcli/midcobra/telemetry.go
@@ -43,7 +43,7 @@ func (m *telemetryMiddleware) postRun(cmd *cobra.Command, args []string, runErr
 	defer telemetry.Stop()
 
 	var userExecErr *usererr.ExitError
-	if errors.As(runErr, &userExecErr) {
+	if errors.As(runErr, &userExecErr) || !usererr.ShouldLogError(runErr) {
 		return
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ func (m telemetryMiddleware) postRun(cmd cobra.Command, args []string, runErr`
`43`	`43`	`defer telemetry.Stop()`
`44`	`44`
`45`	`45`	`var userExecErr *usererr.ExitError`
`46`		`- if errors.As(runErr, &userExecErr) {`
	`46`	`+ if errors.As(runErr, &userExecErr) \|\| !usererr.ShouldLogError(runErr) {`
`47`	`47`	`return`
`48`	`48`	`}`
`49`	`49`