RFR 8133686: REOPEN JDK-8080659 : HttpURLConnection?s getHeaderFields method returns field values in reverse order

Daniel Fuchs daniel.fuchs at oracle.com
Wed Aug 19 09:10:27 UTC 2020


Hi Evan,

On 14/08/2020 13:10, Evan Whelan wrote:
> Hi all,
> 
> I’m Evan and I’ve just recently joined the Java Platform Group here in 
> Oracle so this is my first time getting to interact with the OpenJDK 
> Community.
> 
> This is a request to review my fix for 
> https://bugs.openjdk.java.net/browse/JDK-8133686
> 
> The webrev for this fix can be found at: 
> http://cr.openjdk.java.net/~kravikumar/8133686/webrev.00/webrev/

The fix looks good to me.

> The fix for this bug is straightforward and I don’t believe that this 
> change needs a CSR, as this change makes the source code conform to the 
> RFC2616 spec, and does not introduce any behavioural changes.

Well - there is obviously a behavioral change or there would be
no need for a fix ;-)

I see that you have written a release note.
I don't think a CSR is needed either, but I would welcome a
second opinion too !

best regards,

-- daniel

> 
> That being said, I welcome any differing thoughts!
> 
> I look forward to your responses.
> 
> Thanks,
> 
> Evan
> 



More information about the net-dev mailing list