Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v6]

2023-02-06 Thread Matthew Donovan
> Refactored the code a little bit and updated the test for changes to the TLS > state machine. Matthew Donovan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains eight commits: - fixed conflict - updated SSL context - fixed conflict

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v5]

2023-02-03 Thread Rajan Halade
On Thu, 2 Feb 2023 22:14:13 GMT, Matthew Donovan wrote: >> Refactored the code a little bit and updated the test for changes to the TLS >> state machine. > > Matthew Donovan has updated the pull request incrementally with one > additional commit since the last revision: > > updated SSL conte

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v5]

2023-02-02 Thread Matthew Donovan
> Refactored the code a little bit and updated the test for changes to the TLS > state machine. Matthew Donovan has updated the pull request incrementally with one additional commit since the last revision: updated SSL context - Changes: - all: https://git.openjdk.org/jdk/pull

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v4]

2023-02-02 Thread Matthew Donovan
> Refactored the code a little bit and updated the test for changes to the TLS > state machine. Matthew Donovan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits: - fixed conflict - removed import, updated ProblemList -

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v3]

2023-02-02 Thread Matthew Donovan
> Refactored the code a little bit and updated the test for changes to the TLS > state machine. Matthew Donovan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits: - removed import, updated ProblemList - Merge branch 'mas

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v2]

2023-02-01 Thread Rajan Halade
On Fri, 27 Jan 2023 13:01:01 GMT, Matthew Donovan wrote: >> test/jdk/javax/net/ssl/SSLEngine/EngineCloseOnAlert.java line 67: >> >>> 65: >>> 66: public static void main(String[] args) throws Exception { >>> 67: SecurityUtils.removeFromDisabledTlsAlgs("TLSv1"); >> >> Is this still n

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v2]

2023-01-27 Thread Matthew Donovan
On Thu, 26 Jan 2023 18:53:34 GMT, Rajan Halade wrote: >> Matthew Donovan has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains three commits: >> >> - Merge branch 'master' into engine-close >> - removed unnecessary line >> - 8298868:

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation [v2]

2023-01-27 Thread Matthew Donovan
> Refactored the code a little bit and updated the test for changes to the TLS > state machine. Matthew Donovan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits: - Merge branch 'master' into engine-close - removed unne

Re: RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation

2023-01-26 Thread Rajan Halade
On Mon, 9 Jan 2023 12:19:27 GMT, Matthew Donovan wrote: > Refactored the code a little bit and updated the test for changes to the TLS > state machine. test/jdk/javax/net/ssl/SSLEngine/EngineCloseOnAlert.java line 67: > 65: > 66: public static void main(String[] args) throws Exception { >

RFR: 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation

2023-01-09 Thread Matthew Donovan
Refactored the code a little bit and updated the test for changes to the TLS state machine. - Commit messages: - 8298868: Update EngineCloseOnAlert.java for changes to TLS implementation Changes: https://git.openjdk.org/jdk/pull/11903/files Webrev: https://webrevs.openjdk.org/?rep