Re: [11] RFR: 8194486: Several krb5 tests failed in Mac.

2018-01-10 Thread Weijun Wang
> On Jan 11, 2018, at 1:30 PM, Sibabrata Sahoo > wrote: > > I will remove the commented lines. I think, it should be fine, if I do not > submit the webrev again. Not necessary at all. --Max > > Thanks, > Siba > > -Original Message- > From: Weijun Wang > Sent: Wednesday, January

RE: [11] RFR: 8194486: Several krb5 tests failed in Mac.

2018-01-10 Thread Sibabrata Sahoo
I will remove the commented lines. I think, it should be fine, if I do not submit the webrev again. Thanks, Siba -Original Message- From: Weijun Wang Sent: Wednesday, January 10, 2018 9:07 PM To: Sibabrata Sahoo Cc: [email protected] Subject: Re: [11] RFR: 8194486: Several

About signature algorithm and provider name in SignedObject spec

2018-01-10 Thread Weijun Wang
The class spec of SignedObject.java [1] contains: * {@code * Signature signingEngine = Signature.getInstance(algorithm, * provider); * SignedObject so = new SignedObject(myobject, signingKey, *signingEngine); * } .

Re: Code Review Request, JDK-8185576 : New handshake implementation

2018-01-10 Thread Jamil Nimeh
Hello Xuelei, Here are a few more comments from some of the new files, maybe one or two existing ones that I missed the first time around.  I still need to do the final two big ones (SSLSocketImpl and SSLEngineImpl), but I think I've gone through every other file in the webrev at this point. 

Re: [11] RFR: 8194486: Several krb5 tests failed in Mac.

2018-01-10 Thread Weijun Wang
Code change looks fine. I think you can safely remove the lines commented out. If you meant to keep the history, the Mercurial repo already had it. Thanks Max > On Jan 10, 2018, at 8:58 PM, Sibabrata Sahoo > wrote: > > Hi, > > Please review the change for the following patch, > > JBS: htt

[11] RFR: 8194486: Several krb5 tests failed in Mac.

2018-01-10 Thread Sibabrata Sahoo
Hi, Please review the change for the following patch, JBS: https://bugs.openjdk.java.net/browse/JDK-8194486 Webrev: http://cr.openjdk.java.net/~ssahoo/8194486/webrev.00/ Description: There are several failure occurred recently because the name service used by these Tests were not det

RFR JDK-8194864: Outputs more details for PKCS11 tests if the NSS lib version cannot be determined

2018-01-10 Thread sha . jiang
Hi, JDK-8186098 is raised due to the NSS lib version cannot be determined from the lib content. When this issue occurs, it would be better to output the parsed content for further investigation. Webrev: http://cr.openjdk.java.net/~jjiang/8194864/webrev.00/ Issue: https://bugs.openjdk.java.net/