On Fri, 5 Sep 2025 12:16:48 GMT, Daniel Jeliński <[email protected]> wrote:

>> Fernando Guallini has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   removal of useless tests
>
> test/jdk/javax/net/ssl/DTLS/PacketLossRetransmission.java line 55:
> 
>> 53:  * @run main/othervm PacketLossRetransmission server 2 server_hello
>> 54:  * @run main/othervm PacketLossRetransmission server 3 
>> hello_verify_request
>> 55:  * @run main/othervm -Djdk.tls.client.enableSessionTicketExtension=false 
>> PacketLossRetransmission server 4 new_session_ticket
> 
> This one is odd.
> 
> With `enableSessionTicketExtension=false`, the message is not sent. With the 
> default setting of true, the message is sent, lost, and then the handshake 
> fails with:
> 
> javax.net.ssl.SSLHandshakeException: (decrypt_error) The Finished message 
> cannot be verified.
> 
> This is a bug.
> 
> Could you comment out the line instead? Something like:
> 
> @comment run main/othervm PacketLossRetransmission server 4 new_session_ticket

actually let me file a bug for this...

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27093#discussion_r2329370843

Reply via email to