[9] RFR: 8157107: HTTP/2 client may fail with NPE if additional logging enabled

2016-05-17 Thread Artem Smotrakov
Hello, Please review this patch for 9. NPE may occur if additional logging is enabled with "java.net.http.HttpClient.log" system property because AsyncSSLDelegate.logParams(SSLParameters) doesn't check for null values which were returned by SSLParameters instance. I also noticed that

[OFFLIST] Re: RFR 8016521: InetAddress should not always re-order addresses returned from name service

2016-05-17 Thread Chris Hegarty
[OFFLIST] The code changes, test updates, etc are done. But we need a CCC request, because you are effectively changing a "public interface”, i.e. the set of allowable values for the documented system property. Are you familiar with CCC? http://ccc.us.oracle.com -Chris. On 17 May 2016, at

Re: RFR 8016521: InetAddress should not always re-order addresses returned from name service

2016-05-17 Thread Chris Hegarty
On 17 May 2016, at 14:36, vyom wrote: > Hi Chris, > > Please find the updated > webrev(http://cr.openjdk.java.net/~vtewari/8016521/webrev0.3/index.html > ). i have > did the changes as you suggested.

RFR: 8153142: Cannot resolve multiple values from one response header

2016-05-17 Thread Michael McMahon
Could I get the following change reviewed please? In some circumstances multi-valued response headers were being ignored (except for the first occurrence). The change fixes that problem. It also changes the output of multi-valued request headers to be consistent with that. So instead of sending

Re: RFR 8016521: InetAddress should not always re-order addresses returned from name service

2016-05-17 Thread vyom
Hi Chris, Please find the updated webrev(http://cr.openjdk.java.net/~vtewari/8016521/webrev0.3/index.html ). i have did the changes as you suggested. Thanks, Vyom On Tuesday 17 May 2016 05:04 PM, Chris Hegarty wrote: On

Re: RFR 8016521: InetAddress should not always re-order addresses returned from name service

2016-05-17 Thread Chris Hegarty
On 17 May 2016, at 10:48, vyom wrote: > Hi Chris, > > Please find the updated > webrev(http://cr.openjdk.java.net/~vtewari/8016521/webrev0.2/index.html > ). This looks good Vyom. Thanks. One

Re: RFR 8016521: InetAddress should not always re-order addresses returned from name service

2016-05-17 Thread vyom
Hi Chris, Please find the updated webrev(http://cr.openjdk.java.net/~vtewari/8016521/webrev0.2/index.html ). Thanks, Vyom On Friday 13 May 2016 06:33 PM, Chris Hegarty wrote: Vyom, On 13/05/16 08:23, vyom wrote: Hi,