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

[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 settin