Re: RFR: 8270344: Session resumption errors [v5]

2021-08-20 Thread Sean Coffey
> Corner case where a session resumption can fail if the TLS server changes > supported protocol versions in relation to a cached SSLSession. This is > primarily an issue where the legacy TLS version is used in place of the newer > "supported_versions" TLS extension. Sean Coffey has updated the

Re: RFR: 8270344: Session resumption errors [v5]

2021-08-20 Thread Sean Coffey
On Thu, 19 Aug 2021 19:48:15 GMT, Xue-Lei Andrew Fan wrote: >> Sean Coffey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove redundant method and testcase cleanup > > src/java.base/share/classes/sun/security/ssl/ClientHello.java lin

Re: RFR: 8270344: Session resumption errors [v5]

2021-08-20 Thread Xue-Lei Andrew Fan
On Fri, 20 Aug 2021 16:31:11 GMT, Sean Coffey wrote: >> src/java.base/share/classes/sun/security/ssl/ClientHello.java line 547: >> >>> 545: // handshake output stream, so that the output >>> records >>> 546: // (at the record layer) have the correct versi

Re: RFR: 8270344: Session resumption errors [v5]

2021-08-20 Thread Xue-Lei Andrew Fan
On Fri, 20 Aug 2021 16:34:51 GMT, Sean Coffey wrote: >> Corner case where a session resumption can fail if the TLS server changes >> supported protocol versions in relation to a cached SSLSession. This is >> primarily an issue where the legacy TLS version is used in place of the >> newer "supp