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.

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

force utf8 to save config. #378

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

R-Josef
Copy link

@R-Josef R-Josef commented Jan 28, 2021

I find that every time I restart the server after saving the Chinese configuration file, the configuration file will be garbled. That may be because the configuration file in the jar is utf8, and the platform encode is other, when the plugin saves the default configuration file, it will save utf8 as other encodes.
I thought of two solutions: one is to force the configuration file to be saved in utf8 instead of platform encode; or to read the configuration file in the jar and save it as the platform encode, so that the encode in all places is the platform encode.
This pull request is the first way, i think force utf8 is better than use platform encode, but I also used the second solution in my other branch, if you don't like utf8 then you can consider this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant