RFR: 8255244: HttpClient: Response headers contain incorrectly encoded Unicode characters

[Michael McMahon]

On Wed, 11 Nov 2020 16:45:49 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> The HTTP/1.1 Header Parser of the new HttpClient currently assumes that all headers (names and value) are US-ASCII and as a result mis-decode any byte whose value is > 127; For instance, 0x80 (128) gets decoded as a U+FF80 (65408) instead of being either rejected or decoded as U+0080.
> 
> Historically, HTTP has allowed field content with text in the ISO-8859-1 charset.  The ISO-8859-1 charset is also supported by `HttpURLConnection`.
> 
> We could decide to reject responses whose headers contain non US-ASCII characters out of hand, but for compatibility reasons, it seems preferable to interpret and accept any byte > 127 in header values as an ISO-8859-1 (Latin 1) character.
> For backward compatibility, this change proposes to update the HTTP/1.1 Header Parser to support ISO-8859-1 encoding.
> The HTTP/1.1 Header Parser will now apply the same validation than is already applied by the HTTP/2 stack.

LGTM

-------------

Marked as reviewed by michaelm (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1169