Skip to content

Commit 529882f

Browse files
committed
Check that nil parameters don't get redacted in the -f() variants.
1 parent 2c28eca commit 529882f

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

fmttests/format_error_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,11 @@ func TestFormatViaRedact(t *testing.T) {
3636
sm := string(redact.StartMarker())
3737
em := string(redact.EndMarker())
3838

39-
err := errutil.Newf("hello %s", "world")
39+
var nilErr error
40+
err := errutil.Newf("hello %s %v", "world", nilErr)
4041

41-
tt.CheckEqual(string(redact.Sprintf("%v", err)), `hello `+sm+`world`+em)
42-
tt.CheckEqual(string(redact.Sprintf("%v", errbase.Formattable(err))), `hello `+sm+`world`+em)
42+
tt.CheckEqual(string(redact.Sprintf("%v", err)), `hello `+sm+`world`+em+` <nil>`)
43+
tt.CheckEqual(string(redact.Sprintf("%v", errbase.Formattable(err))), `hello `+sm+`world`+em+` <nil>`)
4344

4445
err = goErr.New("hello")
4546
expected := sm + `hello` + em + `

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ go 1.13
55
require (
66
github.com/cockroachdb/datadriven v1.0.0
77
github.com/cockroachdb/logtags v0.0.0-20190617123548-eb05cc24525f
8-
github.com/cockroachdb/redact v1.0.5
8+
github.com/cockroachdb/redact v1.0.6
99
github.com/cockroachdb/sentry-go v0.6.1-cockroachdb.2
1010
github.com/gogo/protobuf v1.3.1
1111
github.com/gogo/status v1.1.0

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ github.com/cockroachdb/datadriven v1.0.0/go.mod h1:5Ib8Meh+jk1RlHIXej6Pzevx/NLlN
1717
github.com/cockroachdb/errors v1.6.1/go.mod h1:tm6FTP5G81vwJ5lC0SizQo374JNCOPrHyXGitRJoDqM=
1818
github.com/cockroachdb/logtags v0.0.0-20190617123548-eb05cc24525f h1:o/kfcElHqOiXqcou5a3rIlMc7oJbMQkeLk0VQJ7zgqY=
1919
github.com/cockroachdb/logtags v0.0.0-20190617123548-eb05cc24525f/go.mod h1:i/u985jwjWRlyHXQbwatDASoW0RMlZ/3i9yJHE2xLkI=
20-
github.com/cockroachdb/redact v1.0.5 h1:yxqIMS6G2Bvi6GiSHFmsrFGO3aP1rwt8cOm4pixw9eY=
21-
github.com/cockroachdb/redact v1.0.5/go.mod h1:BVNblN9mBWFyMyqK1k3AAiSxhvhfK2oOZZ2lK+dpvRg=
20+
github.com/cockroachdb/redact v1.0.6 h1:W34uRRyNR4dlZFd0MibhNELsZSgMkl52uRV/tA1xToY=
21+
github.com/cockroachdb/redact v1.0.6/go.mod h1:BVNblN9mBWFyMyqK1k3AAiSxhvhfK2oOZZ2lK+dpvRg=
2222
github.com/cockroachdb/sentry-go v0.6.1-cockroachdb.2 h1:IKgmqgMQlVJIZj19CdocBeSfSaiCbEBZGKODaixqtHM=
2323
github.com/cockroachdb/sentry-go v0.6.1-cockroachdb.2/go.mod h1:8BT+cPK6xvFOcRlk0R8eg+OTkcqI6baNH4xAkpiYVvQ=
2424
github.com/codegangsta/inject v0.0.0-20150114235600-33e0aa1cb7c0/go.mod h1:4Zcjuz89kmFXt9morQgcfYZAYZ5n8WHjt81YYWIwtTM=

0 commit comments

Comments
 (0)