RFR: 8349121: SSLParameters.setApplicationProtocols() ALPN example could be clarified [v3]

Artur Barashev abarashev at openjdk.org
Thu Feb 6 19:50:11 UTC 2025


On Thu, 6 Feb 2025 19:35:22 GMT, Artur Barashev <abarashev at openjdk.org> wrote:

>> RFC 7301 says:
>> 
>>     o  Identification Sequence: The precise set of octet values that
>>       identifies the protocol.  This could be the UTF-8 encoding
>>       [[RFC3629](https://datatracker.ietf.org/doc/html/rfc3629)] of the protocol name.
>> 
>> I'll change to `octet byte ordering`.
>
> I still think that `Convert to the default ALPN encoding` comment is less confusing, we are just passing the string in the encoding that the library expects. Basically your comment assumes reader's familiarity with the RFC.

`Convert UTF-8 bytes to the string with default ALPN encoding` also sounds like an option.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23379#discussion_r1945330434


More information about the net-dev mailing list