bcrypt: update InvalidCostError message to describe cost range inclusive

redwrasse · cpu · commit 9dbbcf002b5c · 2025-05-12T11:46:18.000-07:00
Change InvalidCostError string to explicitly state cost bounds range as inclusive, rather than using parentheses that might imply an exclusive range. Change-Id: Ie5e7d53ef4217c8560e6a7fe23e3d65ecc24a2a3 GitHub-Last-Rev: b49002b GitHub-Pull-Request: #300 Reviewed-on: https://go-review.googlesource.com/c/crypto/+/609455 Reviewed-by: Daniel McCarney <daniel@binaryparadox.net> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Cherry Mui <cherryyz@google.com> Reviewed-by: Michael Knyszek <mknyszek@google.com>
diff --git a/bcrypt/bcrypt.go b/bcrypt/bcrypt.go
@@ -50,7 +50,7 @@ func (ih InvalidHashPrefixError) Error() string {
 type InvalidCostError int
 
 func (ic InvalidCostError) Error() string {
-	return fmt.Sprintf("crypto/bcrypt: cost %d is outside allowed range (%d,%d)", int(ic), MinCost, MaxCost)
+	return fmt.Sprintf("crypto/bcrypt: cost %d is outside allowed inclusive range %d..%d", int(ic), MinCost, MaxCost)
 }
 
 const (

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ func (ih InvalidHashPrefixError) Error() string {`
`50`	`50`	`type InvalidCostError int`
`51`	`51`
`52`	`52`	`func (ic InvalidCostError) Error() string {`
`53`		`- return fmt.Sprintf("crypto/bcrypt: cost %d is outside allowed range (%d,%d)", int(ic), MinCost, MaxCost)`
	`53`	`+ return fmt.Sprintf("crypto/bcrypt: cost %d is outside allowed inclusive range %d..%d", int(ic), MinCost, MaxCost)`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`const (`