On Thu, 19 Sep 2024 21:33:11 GMT, Artur Barashev <d...@openjdk.org> wrote:
>> https://bugs.openjdk.org/browse/JDK-8331682 > > Artur Barashev has updated the pull request incrementally with one additional > commit since the last revision: > > Add assertions. Add the final server wrap src/java.base/share/classes/sun/security/ssl/SSLTransport.java line 135: > 133: byte minorVersion = packet.get(); // > pos: 2 > 134: int contentLen = Record.getInt16(packet); // > pos: 3, 4 > 135: Do you think it may be a good idea to check further the content of the alert including the first byte (WARNING or FATAL) and the second byte (AlertDescription for close_notify)? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21043#discussion_r1768862723