Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v20]

2022-05-16 Thread Jaikiran Pai
On Sun, 15 May 2022 06:43:27 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v20]

2022-05-16 Thread Daniel Fuchs
On Sun, 15 May 2022 06:43:27 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v20]

2022-05-16 Thread Michael McMahon
On Sun, 15 May 2022 06:43:27 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v20]

2022-05-16 Thread Jaikiran Pai
On Sun, 15 May 2022 06:43:27 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v20]

2022-05-14 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-11 Thread Jaikiran Pai
On Tue, 10 May 2022 13:51:37 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-11 Thread Daniel Fuchs
On Wed, 11 May 2022 11:34:33 GMT, Michael McMahon wrote: >> Hello Michael, >> Most users will be using the `HttpClient.newBuilder()` to create the >> builder, so this note about `UnsupportedOperationException` can be >> confusing. However, for implementations (libraries?) which provide their

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-11 Thread Michael McMahon
On Wed, 11 May 2022 08:54:48 GMT, Jaikiran Pai wrote: >> src/java.net.http/share/classes/java/net/http/HttpClient.java line 378: >> >>> 376: * >>> 377: * @implSpec The default implementation of this method throws >>> 378: * {@code UnsupportedOperationException}. {@code

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-11 Thread Jaikiran Pai
On Wed, 11 May 2022 08:12:12 GMT, Michael McMahon wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix javadoc link in test > > src/java.net.http/share/classes/java/net/http/HttpClient.java line 378: > >> 376:

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-11 Thread Michael McMahon
On Tue, 10 May 2022 13:51:37 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-10 Thread Daniel Fuchs
On Tue, 10 May 2022 13:51:37 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v18]

2022-05-10 Thread Jaikiran Pai
On Tue, 10 May 2022 13:35:35 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Daniel's review suggestion - add a test to verify the behaviour of the >> localAddress() default method implementati

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v18]

2022-05-10 Thread Jaikiran Pai
On Tue, 10 May 2022 12:37:47 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v19]

2022-05-10 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v18]

2022-05-10 Thread Daniel Fuchs
On Tue, 10 May 2022 13:36:18 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Daniel's review suggestion - add a test to verify the behaviour of the >> localAddress() default method implementati

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v18]

2022-05-10 Thread Daniel Fuchs
On Tue, 10 May 2022 12:37:47 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v17]

2022-05-10 Thread Jaikiran Pai
On Mon, 9 May 2022 13:31:38 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 37 commits: >> >> - Merge latest from master branch >> - add a @build to force jtreg to show consistent test re

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v18]

2022-05-10 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v17]

2022-05-09 Thread Daniel Fuchs
On Mon, 9 May 2022 07:52:29 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specifi

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v17]

2022-05-09 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v15]

2022-04-10 Thread Jaikiran Pai
On Mon, 4 Apr 2022 09:53:02 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specifi

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v16]

2022-04-10 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v15]

2022-04-06 Thread Daniel Fuchs
On Mon, 6 Dec 2021 14:39:27 GMT, Jaikiran Pai wrote: >> Also it's better if the test is IP version agnostic. > > Hello Daniel, I will update the test accordingly in the next round of updates. I'd suggest HttpClientLocalAddrTest implements HttpServerAdapters like other tests do. -

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v15]

2022-04-06 Thread Daniel Fuchs
On Mon, 4 Apr 2022 09:53:02 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specifi

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v14]

2022-04-06 Thread Jaikiran Pai
On Mon, 4 Apr 2022 09:52:56 GMT, Jaikiran Pai wrote: >> src/java.net.http/share/classes/jdk/internal/net/http/HttpClientBuilderImpl.java >> line 140: >> >>> 138: return this; >>> 139: } >>> 140: >> >> Either this should check that localAddr is an `InetSocketAddress` and that >> p

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v14]

2022-03-24 Thread Daniel Fuchs
On Thu, 24 Mar 2022 06:46:01 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v14]

2022-03-23 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v13]

2022-02-28 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v12]

2022-02-21 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v11]

2022-02-14 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v9]

2022-02-03 Thread Jaikiran Pai
On Wed, 2 Feb 2022 10:37:54 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> minor - rename variable in test > > src/java.net.http/share/classes/java/net/http/HttpClient.java line 376: > >> 374:

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v10]

2022-02-03 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v9]

2022-02-02 Thread Daniel Fuchs
On Tue, 7 Dec 2021 14:28:01 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> minor - rename variable in test > > Generally it would be good if the test did not make any assumption on the > presenc

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v9]

2022-02-02 Thread Daniel Fuchs
On Wed, 2 Feb 2022 06:18:39 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specifi

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v9]

2022-02-01 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v9]

2022-02-01 Thread Jaikiran Pai
On Tue, 7 Dec 2021 14:28:01 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> minor - rename variable in test > > Generally it would be good if the test did not make any assumption on the > presenc

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v8]

2022-02-01 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v7]

2022-01-25 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v6]

2022-01-13 Thread Jaikiran Pai
On Wed, 12 Jan 2022 08:48:12 GMT, Jaikiran Pai wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specif

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v6]

2022-01-12 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v5]

2022-01-02 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v4]

2022-01-02 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v3]

2021-12-21 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v2]

2021-12-13 Thread Jaikiran Pai
> This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific local address that will > be used while creating `Socket`

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client

2021-12-10 Thread Jaikiran Pai
On Wed, 8 Dec 2021 01:22:50 GMT, Jaikiran Pai wrote: >> You could also replace `Boolean.getBoolean("java.net.preferIPv6Addresses")` >> with a call to `IPSupport::preferIPv6Addresses()` > >> ::1 is the ipv6 loopback - not the ipv6 wildcard > > Oops, that's embarrassing and it isn't even a copy/p

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client

2021-12-10 Thread Jaikiran Pai
On Tue, 7 Dec 2021 14:35:27 GMT, Daniel Fuchs wrote: >> You may encounter some issues when testing with https and IPv6 - Michael is >> working on a fix to add the loopback addresses as SNI aliases to the >> SimpleSSLContext certificate: >> https://bugs.openjdk.java.net/browse/JDK-8278312 > > Yo

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client

2021-12-10 Thread Daniel Fuchs
On Tue, 7 Dec 2021 14:33:40 GMT, Daniel Fuchs wrote: >> test/jdk/java/net/httpclient/HttpClientLocalAddrTest.java line 205: >> >>> 203: // anyAddress >>> 204: if (Boolean.getBoolean("java.net.preferIPv6Addresses")) { >>> 205: // ipv6 wildcard >> >> ::1 is

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client

2021-12-10 Thread Daniel Fuchs
On Fri, 3 Dec 2021 17:28:25 GMT, Daniel Fuchs wrote: >> This change proposes to implement the enhancement noted in >> https://bugs.openjdk.java.net/browse/JDK-8209137. >> >> The change introduces a new API to allow applications to build a >> `java.net.http.HTTPClient` configured with a specifi

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client

2021-12-10 Thread Jaikiran Pai
On Fri, 3 Dec 2021 17:29:44 GMT, Daniel Fuchs wrote: >> test/jdk/java/net/httpclient/HttpClientLocalAddrTest.java line 74: >> >>> 72: >>> 73: @BeforeClass >>> 74: public static void beforeClass() throws Exception { >> >> For HttpClient tests - unless they are specific to HTTP/1.1 I'd r

Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client

2021-12-10 Thread Daniel Fuchs
On Fri, 3 Dec 2021 06:15:31 GMT, Jaikiran Pai wrote: > This change proposes to implement the enhancement noted in > https://bugs.openjdk.java.net/browse/JDK-8209137. > > The change introduces a new API to allow applications to build a > `java.net.http.HTTPClient` configured with a specific loc