RFR: 8298873: Update IllegalRecordVersion.java for changes to TLS implementation [v2]

Rajan Halade rhalade at openjdk.org
Thu Jan 26 18:54:35 UTC 2023


On Thu, 12 Jan 2023 16:03:34 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> Matthew Donovan has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - renamed IllegalRecordVersion to HandshakeWithInvalidRecordVersion
>>  - Updated IllegalRecordVersion
>>  - reverted change in ProtocolVersion, updated IllegalRecordVersion
>
> test/jdk/javax/net/ssl/SSLEngine/HandshakeWithInvalidRecordVersion.java line 51:
> 
>> 49:     private static final String TRUSTSTORE_PATH =
>> 50:             System.getProperty("test.src", "./") + "/" + PATH_TO_STORES +
>> 51:                     "/" + TRUSTSTORE_FILE;
> 
> It would be nice to avoid to use binary files.  Would you mind to check if test/jdk/javax/net/ssl/templates/SSLContextTemplate.java could be used for the generation of SSLContext (for example test/jdk/javax/net/ssl/templates/SSLEngineTemplate.java)?

There are few other tests that need update as well. We should create backlog item to update all these tests to use the SSLEngineTemplate to generate SSLContext.

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

PR: https://git.openjdk.org/jdk/pull/11929



More information about the security-dev mailing list