RFR [13] 8217429: WebSocket over authenticating proxy fails to send Upgrade headers

Pavel Rappo pavel.rappo at oracle.com
Fri Jan 25 15:16:38 UTC 2019


Chris, thanks for doing this! I have two questions on this change.

1. After this change has been applied, there will be a circular dependency
between HttpRequestImpl and OpeningHandshake. If this code is used by these two
classes maybe we are better off extracting it into some (already existing) third
class?

2. Why does this change add server.close() to each and every test method of
WebSocketTest? If I'm not mistaken that's what @AfterTest public void cleanup()
is supposed to do.

> On 25 Jan 2019, at 14:21, Chris Hegarty <chris.hegarty at oracle.com> wrote:
> 
> When tunneling WebSocket over a proxy requiring authentication, the
> implementation must ensure that the appropriate Upgrade headers are
> not lost after the tunnel has been established. The source changes are
> quite straight forward, the remaining bulk of the changes are to address
> a deficiency in the testing of proxying and authentication, when
> establishing a WebSocket connection.
> 
> Webrev:
>  http://cr.openjdk.java.net/~chegar/8217429/webrev.01/
> Bug:
>  https://bugs.openjdk.java.net/browse/JDK-8217429
> 
> -Chris



More information about the net-dev mailing list