On Fri, 5 Nov 2021 17:23:40 GMT, Jamil Nimeh <jni...@openjdk.org> wrote:

>> This fix handles cases where two or more threads may be acting on a single 
>> SSLSocket instance.  In cases where one thread closes the socket while the 
>> other is in the middle of a read, a SocketException is appropriately thrown, 
>> but it results in SSLSession invalidation even though the session was 
>> legitimately negotiated at an earlier point in time and should remain in the 
>> cache.
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8274736
>
> Jamil Nimeh has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Do not reinstantiate SSLContext, add more iterations, add test batch 
> without using session tickets

test/jdk/sun/security/ssl/SSLSessionImpl/NoInvalidateSocketException.java line 
34:

> 32:  * @test
> 33:  * @bug 8274736
> 34:  * @summary Concurrent read/close of SSLSockets causes SSLSessions to be 
> invalidated unnecessarily

Minor nit.  Can you keep these under 80 chars?

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

PR: https://git.openjdk.java.net/jdk/pull/6197

Reply via email to