RFR[9] JDK-8170641: sun/net/www/protocol/https/HttpsURLConnection/PostThruProxy.sh fails with timeout

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR[9] JDK-8170641: sun/net/www/protocol/https/HttpsURLConnection/PostThruProxy.sh fails with timeout

sha.jiang
Hi,
sun/net/www/protocol/https/HttpsURLConnection/PostThruProxy.sh failed
with timeout.
The client side could hang if the server goes to timeout before getting
the client request, and the proxy also could be blocked.

This patch sets timeout for the server and the client and catches
SocketTimeoutException to ignore the . And it removes the usage on shell
script.
The patch also modifies test
sun/net/www/protocol/https/HttpsURLConnection/PostThruProxyWithAuth.sh
as this test has the similar code pattern and issue.

Webrev: http://cr.openjdk.java.net/~jjiang/8170641/webrev.00/
Issue: https://bugs.openjdk.java.net/browse/JDK-8170641

Best regards,
John Jiang

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR[9] JDK-8170641: sun/net/www/protocol/https/HttpsURLConnection/PostThruProxy.sh fails with timeout

Chris Hegarty
On 27 Dec 2016, at 06:24, John Jiang <[hidden email]> wrote:

>
> Hi,
> sun/net/www/protocol/https/HttpsURLConnection/PostThruProxy.sh failed with timeout.
> The client side could hang if the server goes to timeout before getting the client request, and the proxy also could be blocked.
>
> This patch sets timeout for the server and the client and catches SocketTimeoutException to ignore the . And it removes the usage on shell script.
> The patch also modifies test sun/net/www/protocol/https/HttpsURLConnection/PostThruProxyWithAuth.sh as this test has the similar code pattern and issue.
>
> Webrev: http://cr.openjdk.java.net/~jjiang/8170641/webrev.00/
> Issue: https://bugs.openjdk.java.net/browse/JDK-8170641

Looks good John.

Thanks,
-Chris.
Loading...