On Mon, 13 Oct 2025 13:19:59 GMT, Daniel Jeliński <[email protected]> wrote:
>> Artur Barashev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> The fix and unit test for session resumption case > > test/jdk/javax/net/ssl/DTLS/DTLSOverDatagram.java line 138: > >> 136: isClient ? getClientDTLSContext() : >> getServerDTLSContext(); >> 137: SSLEngine engine = context.createSSLEngine(HOST, >> 138: isClient ? CLIENT_PORT : SERVER_PORT); > > Suggestion: > > isClient ? SERVER_PORT : CLIENT_PORT); > > Client uses server port, server uses client port. Not sure what I'm missing here: if `isClient` is `true` then we use `CLIENT_PORT` ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27677#discussion_r2426444171
