Hi Evan,

Thanks for the new test.
Could you also add an additional step that transmits the
request properties to the server and checks that they are
received by the remote side in the expected order?

Also can you use try { } finally { } to close the HttpServer?


best regards,

-- daniel

On 03/09/2020 10:25, Evan Whelan wrote:
Hi,

Thanks for the feedback Daniel, I've added the test verifying the issue.

The test checks both getHeaderFields and getRequestProperties.

Please find another revision of this fix 
at:http://cr.openjdk.java.net/~kravikumar/8133686/webrev.03/
A CSR for this issue has also been written and is found 
at:https://bugs.openjdk.java.net/browse/JDK-8252456

Please let me know if you have any more suggestions or feedback.

Kind regards,
Evan

Reply via email to