mirrors
/
forgejo
mirror of
1
Fork 0
Code Issues Releases Activity

Fix `cases.Title` crash for concurrency (#23885) (#23903) 

Backport #23885 by @wxiaoguang

Regression of #19676 and #21814

Fix #23872

`cases.Title` is not thread-safe, it has internal state, so it can't be
used as a global shared variable.

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
This commit is contained in:
Giteabot 2023-04-03 19:26:14 -04:00 committed by GitHub
parent ac57ec5c47
commit 669c76c0fd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
modules/util/util.go
View File

@ -185,19 +185,16 @@ func ToUpperASCII(s string) string {
return string(b) return string(b)
} }
var (
titleCaser = cases.Title(language.English)
titleCaserNoLower = cases.Title(language.English, cases.NoLower)
)
// ToTitleCase returns s with all english words capitalized // ToTitleCase returns s with all english words capitalized
func ToTitleCase(s string) string { func ToTitleCase(s string) string {
return titleCaser.String(s) // `cases.Title` is not thread-safe, do not use global shared variable for it
return cases.Title(language.English).String(s)
} }
// ToTitleCaseNoLower returns s with all english words capitalized without lowercasing // ToTitleCaseNoLower returns s with all english words capitalized without lower-casing
func ToTitleCaseNoLower(s string) string { func ToTitleCaseNoLower(s string) string {
return titleCaserNoLower.String(s) // `cases.Title` is not thread-safe, do not use global shared variable for it
return cases.Title(language.English, cases.NoLower).String(s)
} }
var ( var (