scrub phone number from failed SMS log line (#2119)

mikehelmick · web-flow · commit 08eefedecc4a · 2021-06-04T21:25:14.000Z
diff --git a/pkg/controller/userreport/send.go b/pkg/controller/userreport/send.go
@@ -169,7 +169,7 @@ func (c *Controller) HandleSend() http.Handler {
 			}
 
 			if !suppressError {
-				logger.Errorw("unable to issue user-report code", "status", result.HTTPCode, "error", result.ErrorReturn.Error)
+				logger.Errorw("unable to issue user-report code", "status", result.HTTPCode, "error", issueapi.ScrubPhoneNumbers(result.ErrorReturn.Error))
 				// The error returned isn't something the user can easily fix, show internal error, but hide form.
 				m["error"] = []string{locale.Get("user-report.internal-error")}
 				m["skipForm"] = true

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ func (c *Controller) HandleSend() http.Handler {`
`169`	`169`	`}`
`170`	`170`
`171`	`171`	`if !suppressError {`
`172`		`- logger.Errorw("unable to issue user-report code", "status", result.HTTPCode, "error", result.ErrorReturn.Error)`
	`172`	`+ logger.Errorw("unable to issue user-report code", "status", result.HTTPCode, "error", issueapi.ScrubPhoneNumbers(result.ErrorReturn.Error))`
`173`	`173`	`// The error returned isn't something the user can easily fix, show internal error, but hide form.`
`174`	`174`	`m["error"] = []string{locale.Get("user-report.internal-error")}`
`175`	`175`	`m["skipForm"] = true`