On Mon, 21 Mar 2022 15:05:02 GMT, Sean Coffey <coff...@openjdk.org> wrote:

>> JDK-8253368 changed the behavior of SSLSocket to no longer throw a fatal 
>> internal_error (80) and invalidate existing sessions (either completed or 
>> under construction) as described in (RFC 4346/TLSv1.1+) if a connection was 
>> closed without receiving a close_notify alert from the peer.  
>> 
>> This change introduces similar behavior to SSLEngine.
>> 
>> The unit test checks that closing the read(input) sides of the 
>> SSLSocket/SSLEngine throws an SSLException, but doesn't invalidate their 
>> respective sessions.
>> 
>> Tier1/2 mach5 tests have been successfully run.
>
> test/jdk/sun/security/ssl/SSLSocketImpl/SSLSocketSSLEngineCloseInbound.java 
> line 130:
> 
>> 128:      * The following is to set up the keystores/trust material.
>> 129:      */
>> 130:     private static final String pathToStores = 
>> "../../../../javax/net/ssl/etc";
> 
> I think the advise is to move away from binary style keystores. i.e. to use 
> test/jdk/javax/net/ssl/templates/SSLContextTemplate.java for a replacement. 
> Is that possible here ?

Sigh...this is a whole can of worms I wasn't expecting.  Looks like one person 
did the SSLContextTemplate and updated with SSLEngineTemplate, then another 
person took a completely different takes with SSLSocketTemplate, and then 
SSLSocketSSLEngineTemplate didn't get touched at all.  

This should really be harmonized.  :(

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

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

Reply via email to