Re: RFR 8005819: Support cross-realm MSSFU

2019-12-12 Thread Weijun Wang
> On Dec 12, 2019, at 10:46 PM, Martin Balao wrote: > > Good observation. I've had a look and you are probably right. The in/out > parameter is spread across all the code. Looks to me that, at some > point, the idea was that there could be more than one additional ticket > (and it was not an i

Re: [14]RFR: 8234728: Some security tests should support TLSv1.3

2019-12-12 Thread Xuelei Fan
Looks good to me. Thanks, Xuelei On 12/12/2019 2:25 AM, [email protected] wrote: Hi Xuelei, Please review the patch for, JBS: https://bugs.openjdk.java.net/browse/JDK-8234728 Webrev: http://cr.openjdk.java.net/~ssahoo/8234728/webrev.00/ Change: The change it to support few of Tests t

Re: RFR 8005819: Support cross-realm MSSFU

2019-12-12 Thread Martin Balao
On 12/11/19 10:07 PM, Weijun Wang wrote: > This looks good to me. Great! > > There is one confusion. I understand that handleS4U2ProxyReferral needs an > in/out creds argument, but for serviceCredsReferrals the additionalTickets > argument should only be in, right? However in this method you'v

[14]RFR: 8234728: Some security tests should support TLSv1.3

2019-12-12 Thread [email protected]
Hi Xuelei, Please review the patch for, JBS: https://bugs.openjdk.java.net/browse/JDK-8234728 Webrev: http://cr.openjdk.java.net/~ssahoo/8234728/webrev.00/ Change: The change it to support