RFR JDK-8191918: tomcat gzip-compressed response bodies appear to be broken in update 151

[Seán Coffey]

Looks good to me.

Regards,
Sean.

On 02/12/17 01:04, Xueming Shen wrote:
> On 12/1/17, 3:40 PM, Paul Sandoz wrote:
>>
>>> On 30 Nov 2017, at 14:46, Xueming Shen<xueming.shen at oracle.com>  wrote:
>>>
>>> Hi,
>>>
>>> Please help review the change for  JDK-8191918:
>>>
>>> issue: https://bugs.openjdk.java.net/browse/JDK-8191918
>>> webrev: http://cr.openjdk.java.net/~sherman/8191918/webrev
>>>
>> InflateIn_DeflateOut.java
>> —
>>
>>   174         GZIPInputStream gzis = new GZIPInputStream(byteInStream);
>>   175         ByteArrayOutputStream baos = new ByteArrayOutputStream();
>>   176         int numRead;
>>   177         byte[] b = new byte[4 * 1024];
>>   178         try {
>>   179             while ((numRead = gzis.read(buf))>= 0) {
>>   180                 baos.write(b, 0, numRead);
>>   181             }
>>   182         } finally {
>>   183             baos.close();
>>   184         }
>>
>> Can you use gzis.transferTo(baos)?
>>
>> Paul.
>>
>>
>
> Thanks Paul!
>
> webrev has been updated as suggested.
>
> http://cr.openjdk.java.net/~sherman/8191918/webrev
>
> -Sherman