Opened 2 years ago

Closed 2 years ago

#25740 closed defect (fixed)

Provide utf8 character instead of escape sequence

Reported by: cypherpunks Owned by: iwakeh
Priority: Medium Milestone: Onionoo 1.13.0
Component: Metrics/Onionoo Version:
Severity: Normal Keywords:
Cc: metrics-team Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Since the fix for #22594 has been deployed UTF is no longer shown as previously.

Example:
it used to be:
TELEFÔNICA BRASIL S.A

after the deployment of #22594:
TELEF\u00D4NICA BRASIL S.A

Child Tickets

Change History (5)

comment:1 Changed 2 years ago by iwakeh

Cc: metrics-team added
Owner: changed from metrics-team to iwakeh
Status: new → accepted

comment:2 Changed 2 years ago by iwakeh

Summary: #22594 introduces regression for UTF strings → Provide utf8 character instead of escape sequence

Adapting the summary.

comment:3 Changed 2 years ago by iwakeh

Status: accepted → needs_review

Thanks for reporting this issue immediately!

This wasn't really a regression, but rather a step missing, i.e., valid utf escape sequences should be turned into utf characters in addition. Please review ​this patch branch.

comment:4 Changed 2 years ago by iwakeh

Milestone: → Onionoo 1.13.0

comment:5 Changed 2 years ago by karsten

Resolution: → fixed
Status: needs_review → closed

Merged together with a change log entry. Closing. Thanks!

Note: See TracTickets for help on using tickets.