Re: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-04-07 Thread Seán Coffey
wrote: Thanks for review Xuelei, I will incorporate your suggestions. Thanks, Prasad.K -Original Message- From: Xuelei Fan Sent: Thursday, April 2, 2020 9:12 PM To: security-dev@openjdk.java.net Subject: Re: RFR[jdk] 8237474: Default SSLEngine should create in server role Please upda

RE: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-04-02 Thread Prasadrao Koppula
Thanks for review Xuelei, I will incorporate your suggestions. Thanks, Prasad.K > -Original Message- > From: Xuelei Fan > Sent: Thursday, April 2, 2020 9:12 PM > To: security-dev@openjdk.java.net > Subject: Re: RFR[jdk] 8237474: Default SSLEngine should create in server r

Re: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-04-02 Thread Xuelei Fan
curity-dev@openjdk.java.net *Subject:* RFR[jdk] 8237474: Default SSLEngine should create in server role Hi, Could you please review this patch. Default server role mode was flipped in SSLEngine, to client role mode as part of SSL package code refactoring for TLSv1.3, this patch flips back default clie

RE: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-04-01 Thread Prasadrao Koppula
Gentle ping? Thanks, Prasad.K From: Prasadrao Koppula Sent: Monday, March 30, 2020 7:21 PM To: security-dev@openjdk.java.net Subject: RE: RFR[jdk] 8237474: Default SSLEngine should create in server role Hi, Added @implnote and updated test changes, here is the new webrev, please

RE: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-03-30 Thread Prasadrao Koppula
, Prasad.K From: Prasadrao Koppula Sent: Friday, February 7, 2020 5:03 PM To: security-dev@openjdk.java.net Subject: RFR[jdk] 8237474: Default SSLEngine should create in server role Hi, Could you please review this patch. Default server role mode was flipped in SSLEngine, to client role mode

RE: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-02-10 Thread Prasadrao Koppula
> -Original Message- > From: Daniel Fuchs > Sent: Friday, February 7, 2020 8:32 PM > To: Prasadrao Koppula ; Sean Coffey > ; security-dev@openjdk.java.net > Subject: Re: RFR[jdk] 8237474: Default SSLEngine should create in server role > > Hi Prasad, > >

Re: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-02-07 Thread Daniel Fuchs
Hi Prasad, On 07/02/2020 14:28, Prasadrao Koppula wrote: Thanks for review Sean, I will add test changes. Not a review - but I just wanted to double check that you have run the :jdk_net tests too - especially the httpclient tests (which are part of :jdk_net) as the httpclient is a heavy user o

RE: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-02-07 Thread Prasadrao Koppula
Thanks for review Sean, I will add test changes.   Thanks, Prasad.K   From: Seán Coffey Sent: Friday, February 7, 2020 6:20 PM To: Prasadrao Koppula ; security-dev@openjdk.java.net Subject: Re: RFR[jdk] 8237474: Default SSLEngine should create in server role   Looks ok to me Prasad. This

Re: RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-02-07 Thread Seán Coffey
Looks ok to me Prasad. This may also be worthy of highlighting via release note. You might be able to expand test coverage to capture the TLSContext scenario. Something like below patch might work ? --- a/test/jdk/sun/security/ssl/SSLEngineImpl/EngineEnforceUseClientMode.java +++ b/test/jdk/

RFR[jdk] 8237474: Default SSLEngine should create in server role

2020-02-07 Thread Prasadrao Koppula
Hi, Could you please review this patch. Default server role mode was flipped in SSLEngine, to client role mode as part of SSL package code refactoring for TLSv1.3, this patch flips back default client role to server role in SSLEngine. webrev: http://cr.openjdk.java.net/~pkoppula/8237474/w