Closed Bug 964113 Opened 7 years ago Closed 7 years ago

Digest::SHA 5.82 and newer always croak on wide characters, preventing saved searched from being displayed

Categories

(Bugzilla :: Query/Bug List, defect)

4.4.1
defect
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

(Keywords: regression)

Attachments

(1 file)

Digest::SHA no longer accepts Unicode characters since 5.82, preventing saved searches from being displayed if their name contains such characters. The fix I implemented in bug 477513 has been reverted in bug 767623 in 4.4 because at that time it was fine to pass Unicode characters (5.82 wasn't released yet). This fix must come back.
Flags: blocking4.4.2+
Attached patch patch, v1Splinter Review
Must land before we release 4.4.2.
Attachment #8365738 - Flags: review?(justdave)
Attachment #8365738 - Flags: review?(dkl)
I did some testing, and my patch restores exactly the old behavior, i.e. passing the same Unicode string returns exactly the same key as before 5.82. So "old" tokens won't be invalidated by this patch, which is good. :)
Comment on attachment 8365738 [details] [diff] [review]
patch, v1

r=dkl
Attachment #8365738 - Flags: review?(dkl) → review+
Flags: approval?
Flags: approval4.4?
Attachment #8365738 - Flags: review?(justdave)
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Token.pm
modified template/en/default/pages/release-notes.html.tmpl
Committed revision 8894.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
modified Bugzilla/Token.pm
modified template/en/default/pages/release-notes.html.tmpl
Committed revision 8652.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.