RFR: 8372526: Add support for ZLIB TLS Certificate Compression [v17]
Artur Barashev
abarashev at openjdk.org
Wed Feb 11 09:40:58 UTC 2026
On Tue, 10 Feb 2026 21:23:26 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> Artur Barashev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Correct message name in javadoc
>
> src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 969:
>
>> 967: * This method only applies to TLSv1.3.
>> 968: *
>> 969: * @apiNote The peer needs to support the underlying extension
>
> Instead of "underlying", say the specific extension so it's clear what needs to be supported, i.e. "compress_certificate extension".
>
> Also, I would change "needs to" to "must".
Done, thanks!
> src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 973:
>
>> 971: * to work.
>> 972: *
>> 973: * @implNote The SunJSSE provider supports only zlib compression.
>
> s/supports only/only supports/ (sounds better)
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28682#discussion_r2790600784
PR Review Comment: https://git.openjdk.org/jdk/pull/28682#discussion_r2790601510
More information about the security-dev
mailing list