> On 25 Jan 2019, at 17:04, Chris Hegarty wrote:
>
> I moved the code to common.Utils, to avoid any unnecessary dependency.
Thanks.
>> 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
>>
Thanks Chris.
Makes sense, and looks good!
best regards,
-- daniel
On 25/01/2019 17:07, Chris Hegarty wrote:
Daniel,
On 25 Jan 2019, at 15:34, Daniel Fuchs wrote:
Hi Chris,
Looks good.
I had the same question than Pavel about server.close().
Answered already in reply to Pavel’s
Daniel,
> On 25 Jan 2019, at 15:34, Daniel Fuchs wrote:
>
> Hi Chris,
>
> Looks good.
> I had the same question than Pavel about server.close().
Answered already in reply to Pavel’s question.
> No test for both proxy + server authorization with
> -Djdk.http.auth.tunneling.disabledSchemes ?
Pavel,
> On 25 Jan 2019, at 15:16, Pavel Rappo 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
>
Hi Chris,
Looks good.
I had the same question than Pavel about server.close().
No test for both proxy + server authorization with
-Djdk.http.auth.tunneling.disabledSchemes ?
cheers,
-- daniel
On 25/01/2019 14:21, Chris Hegarty wrote:
When tunneling WebSocket over a proxy requiring
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)
> On 25 Jan 2019, at 14:10, Daniel Fuchs wrote:
>
> Hi,
>
> Please find below a doc clarification change for:
>
> 8217627: HttpClient: The API documentation of BodySubscribers::mapping
> promotes bad behavior
> https://bugs.openjdk.java.net/browse/JDK-8217627
>
> webrev:
>
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
Hi,
Please find below a doc clarification change for:
8217627: HttpClient: The API documentation of BodySubscribers::mapping
promotes bad behavior
https://bugs.openjdk.java.net/browse/JDK-8217627
webrev:
http://cr.openjdk.java.net/~dfuchs/webrev_8217627/webrev.00
New API doc:
For
I've created a branch in the sandbox, named "niosocketimpl-branch", with
a prototype SocketImpl implementation based on the infrastructure in
sun.nio.ch package that supports the NIO channel implementations. The
branch also includes the changes to java.net.Socket and ServerSocket to
use
10 matches
Mail list logo
