Re: [teststabilization] RFR: 8226514: Replace wildcard address with loopback or local host in tests - part 17

2019-06-26 Thread Vyom Tewari26
Looks good to me Daniel. Vyom   - Original message -From: Daniel Fuchs Sent by: "net-dev" To: OpenJDK Network Dev list Cc:Subject: [EXTERNAL] [teststabilization] RFR: 8226514: Replace wildcard address with loopback or local host in tests - part 17Date: Wed, Jun 26, 2019 9:00 PM  Hi,Please f

Re: [teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-26 Thread Chris Hegarty
Julia, > On 26 Jun 2019, at 18:33, Julia Boes wrote: > > Hi, > > Please find below a patch for: > > 8226825: Replace wildcard address with loopback or local host in tests - part > 19 > > https://bugs.openjdk.java.net/browse/JDK-8226825 > > > webrev: > > http://cr.openjdk.java.net/~dfuchs/

Re: [teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-26 Thread Brian Burkhalter
Hi Julia, (part 19 (!)) In AsyncDisconnect line 57 could be replaced with ioe.printStackTrace() to be consistent with lines 85 and 88. Same thing for B6641309 line 58, and B6660405 line 108. I’ll leave comments on the more substantive changes to the net-dev experts. Thanks, Brian > On Jun 26

[teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-26 Thread Julia Boes
Hi, Please find below a patch for: 8226825: Replace wildcard address with loopback or local host in tests - part 19 https://bugs.openjdk.java.net/browse/JDK-8226825 webrev: http://cr.openjdk.java.net/~dfuchs/jboes/webrev_8226825/webrev/ - Replaced wildcard address with loopback or localho

[teststabilization] RFR: 8226514: Replace wildcard address with loopback or local host in tests - part 17

2019-06-26 Thread Daniel Fuchs
Hi, Please find below a patch for: 8226514: Replace wildcard address with loopback or local host in tests - part 17 https://bugs.openjdk.java.net/browse/JDK-8226514 webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8226514/webrev.00/ - The SetAuthenticator/ test had already been upda