Skip to content

Replace Charset.defaultCharset() with StandardCharsets.UTF_8 #4418

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
Closed
wants to merge 1 commit into from
Closed

Replace Charset.defaultCharset() with StandardCharsets.UTF_8 #4418

wants to merge 1 commit into from

Conversation

lsq27
Copy link
Contributor

@lsq27 lsq27 commented Jul 15, 2023

Replace Charset.defaultCharset() with StandardCharsets.UTF_8

fix #4417

@fmbenhassine
Copy link
Contributor

LGTM! Rebased and merged as 14b6c66. Thank you for raising the issue and for contributing a fix 👍

@lsq27 lsq27 deleted the GH-4417 branch July 20, 2023 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
in: build pr-for: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Build failure when the default charset of the JVM is not UTF-8
2 participants
@lsq27 @fmbenhassine