Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: ensure LinkSanitizer returns utf-8 encoded strings #163

Merged
merged 1 commit into from May 18, 2023
Merged

fix: ensure LinkSanitizer returns utf-8 encoded strings #163

merged 1 commit into from May 18, 2023

Conversation

flavorjones
Copy link
Member

Previously, 2362298 ensured utf-8 encoded values were returned from SafeListSanitizer, and 49dfc15 continued this practice for FullSanitizer.

At no point was this behavior added to LinkSanitizer, which is fixed by this commit.

@flavorjones
fix: ensure LinkSanitizer returns UTF-8 encoded strings
47f6255 
Previously, 2362298 ensured SafeListSanitizer returned UTF-8 encoded
strings, and 49dfc15 introduced FullSanitizer which did the same.

This behavior is now being added to the remaining sanitizer,
LinkSanitizer.
@flavorjones flavorjones force-pushed the flavorjones-ensure-utf8-encoding-from-all-sanitizers branch from 714c77d to 47f6255 Compare May 18, 2023 01:35
@flavorjones flavorjones merged commit b14f89d into main May 18, 2023
13 checks passed
@flavorjones flavorjones deleted the flavorjones-ensure-utf8-encoding-from-all-sanitizers branch May 18, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rafaelfranca rafaelfranca rafaelfranca approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@flavorjones @rafaelfranca