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

2016-05-18 Thread Michael McMahon
Looks good, Artem. Thanks, Michael On 17/05/16 22:27, Artem Smotrakov wrote: 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 va

RFR 8157105: HTTP/2 client hangs in blocking mode if an invalid frame has been received

2016-05-18 Thread Michael McMahon
Could I get the following change reviewed please? http://cr.openjdk.java.net/~michaelm/8157105/webrev.1/ Some error conditions not being handled correctly and some logging cleanup. Thanks Michael.

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

2016-05-18 Thread Roger Riggs
Hi Michael, Looks fine. Roger On 5/17/2016 9:43 AM, Michael McMahon wrote: 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