Re: RFR: 8284780: Need methods to create pre-sized HashSet and LinkedHashSet [v17]

2022-06-02 Thread Daniel Fuchs
On Fri, 27 May 2022 18:40:32 GMT, XenoAmess wrote: >> as title. > > XenoAmess has updated the pull request incrementally with one additional > commit since the last revision: > > do it as naotoj said Changes to `net` and `http` look good. - Marked as reviewed by dfuchs (Reviewe

Re: RFR: 8286779: javax.crypto.CryptoPolicyParser#isConsistent always returns 'true'

2022-06-02 Thread Sean Mullan
On Thu, 2 Jun 2022 01:53:25 GMT, Hai-May Chao wrote: > Please review a small fix in CryptoPolicyParser class that it should not pass > “processedPermissions” parameter by value. > Ran MACH5 tier1 and tier2 without failures. The fix looks fine. Could you add a test for this in the `test/jdk/jav

Re: RFR: 8277307: Pre shared key sent under both session_ticket and pre_shared_key extensions

2022-06-02 Thread Daniel Jeliński
On Tue, 31 May 2022 13:47:28 GMT, Sean Coffey wrote: >> Session ticket extension should only contain pre-TLS1.3 stateless session >> tickets; it should not be used for sending TLS1.3 pre-shared keys. > > src/java.base/share/classes/sun/security/ssl/SessionTicketExtension.java line > 410: > >>

Re: RFR: 8277307: Pre shared key sent under both session_ticket and pre_shared_key extensions [v2]

2022-06-02 Thread Daniel Jeliński
> Session ticket extension should only contain pre-TLS1.3 stateless session > tickets; it should not be used for sending TLS1.3 pre-shared keys. Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision: different check for TLS13 --

Re: RFR: 8287596: Reorg jdk.test.lib.util.ForceGC [v4]

2022-06-02 Thread Xue-Lei Andrew Fan
> This is a follow up update per comments in [JDK-8287384 > PR](https://github.com/openjdk/jdk/pull/8907). The tier1 and tier2 test in > open part looks good to me. Please help to run Mach5 just case the closed > test cases are impacted. Xue-Lei Andrew Fan has updated the pull request increme