RFR: 8353698: Output of Simple Web Server is garbled if the console's encoding is not UTF-8 [v3]
Daniel Fuchs
dfuchs at openjdk.org
Fri Apr 4 12:33:07 UTC 2025
On Fri, 4 Apr 2025 09:13:30 GMT, Daishi Tabata <duke at openjdk.org> wrote:
>> The output `jwebserver` and `java -m jdk.httpserver` uses UTF-8 encoding. Therefore, if the console encoding is not set to UTF-8 (for example, MS932 on Japanese Windows), garbled characters may appear.
>> Since System.out knows the console's encoding, there's no need to specify it again with PrintWriter constructor.
>>
>> The output `jwebserver` and `java -m jdk.httpserver` is localized in Japanese, German, and Chinese. This bug only occurs in these localized languages, so I haven't added any tests.
>> I have confirmed that the output of the two commands is not garbled in the default command prompts of both Japanese Windows and English Windows.
>>
>> I tested jdk/com/sun/net/httpserver/simpleserver on English Windows, and all tests passed. However, when running on Japanese Windows, some tests failed because they expected output only in English.
>> I modified the tests so that `jwebserver` and `java -m jdk.httpserver` within the tests are executed with an English locale. After this modification, all tests passed on Japanese Windows.
>
> Daishi Tabata has updated the pull request incrementally with one additional commit since the last revision:
>
> Update full name
> Upon reconsideration, these tests were failing on Japanese Windows even before the fix.
> Should I issue a separate Bug ID?
That's what I suspected. I just wanted to make sure that needing the locale was not a side effect of your changes.
That said - I believe our tests are typically expecting the CI system (or make)? to set Locale.ENGLISH before running the tests.
I'm good with the changes you're proposing. Please hold on integrating for a bit - I would like to verify that there no regression in our CI.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24437#issuecomment-2778586005
More information about the net-dev
mailing list