[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-net-dev
Subject:    Re: RFR: 8209137: Add ability to bind to specific local address to HTTP client [v3]
From:       Jaikiran Pai <jpai () openjdk ! java ! net>
Date:       2021-12-21 13:04:08
Message-ID: HS9qWA3923ErKp6VBcSvr81G-pDqiT6SjTWT9iWh8ro=.b0cbc23b-a370-4745-b6ff-15d683ce6549 () github ! com
[Download RAW message or body]

> 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`(s) for connections.

Jaikiran Pai has updated the pull request with a new target base due to a merge or a \
rebase. The incremental webrev excludes the unrelated changes brought in by the \
merge/rebase. The pull request contains 16 additional commits since the last \
revision:

 - Merge latest from master branch
 - add a security manager test to verify proper permission checks happen when local \
                address is configured on HTTPClient
 - Merge latest from master branch
 - add a note to the javadoc of the new API to explain that calling localAddress() is \
                only for advanced usages
 - move the security checks to the HttpClient itself instead of the builder
 - first round of security manager checks
 - use IPSupport in test and don't rely on ipv4/ipv6 being available
 - use correct IPv6 "any address"
 - Refactor the test to use the HttpServerAdapters test infrastructure
 - increase the jtreg maxOutputSize for java/net/httpclient tests
 - ... and 6 more: https://git.openjdk.java.net/jdk/compare/ed48d3ac...b7a894c4

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6690/files
  - new: https://git.openjdk.java.net/jdk/pull/6690/files/9107b2ac..b7a894c4

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=6690&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=6690&range=01-02

  Stats: 13092 lines in 335 files changed: 9025 ins; 2829 del; 1238 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6690.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6690/head:pull/6690

PR: https://git.openjdk.java.net/jdk/pull/6690


[prev in list] [next in list] [prev in thread] [next in thread]