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

Chris Hegarty chris.hegarty at oracle.com
Fri Jan 25 17:04:16 UTC 2019


Pavel,

> On 25 Jan 2019, at 15:16, Pavel Rappo <pavel.rappo at oracle.com> wrote:
> 
> 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?

I moved the code to common.Utils, to avoid any unnecessary dependency.

> 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.

I think @AfterTest does not do what you think it does.

@AfterTest: The annotated method will be run after all the test methods
belonging to the classes inside the <test> tag have run.

Really, these tests should use try-with-resources, but I wanted avoid
obfuscating the changes. That can be done separately.

-Chris.


More information about the net-dev mailing list