RFR: 7065228: To interpret case-insensitive string locale independently
Daniel Jeliński
djelinski at openjdk.org
Wed May 17 10:44:47 UTC 2023
On Tue, 16 May 2023 10:38:52 GMT, Darragh Clarke <duke at openjdk.org> wrote:
> Updated instances of `toLowerCase` and `toUpperCase` in several net and io files to specify `Locale.ROOT` to ensure that case conversion issues don't occur,
>
> I didn't add any new tests but ran tier 1-3 with no issues
src/java.base/share/classes/sun/net/www/protocol/http/DigestAuthentication.java line 516:
> 514:
> 515: if (nccount != -1) {
> 516: ncstring = Integer.toHexString (nccount).toLowerCase(Locale.ROOT);
Suggestion:
ncstring = Integer.toHexString(nccount);
`toHexString` returns lowercase string
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14006#discussion_r1196296029
More information about the core-libs-dev
mailing list