RFR: 8252304: Seed an HttpRequest.Builder from an existing HttpRequest [v8]
Patrick Concannon
pconcannon at openjdk.java.net
Mon Nov 16 15:54:18 UTC 2020
> Hi,
>
> Could someone please review our code for JDK-8252304: 'Seed an HttpRequest.Builder from an existing HttpRequest'?
>
> This RFR proposes a new factory method for creating a new `HttpRequest` builder from an existing `HttpRequest`.
> This method can be used to build a new request equivalent to the given request, but with different attributes. For instance, it will allow the user to take an existing request and add or change a particular header, provide a new `URI`, etc.
>
>
> Kind regards,
> Patrick & Chris
Patrick Concannon 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 15 additional commits since the last revision:
- 8252304: Added more test cases to check header filter
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- 8252304: BiPredicate parameter added to newBuilder
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- Merge remote-tracking branch 'origin/master' into JDK-8252304
- 8252304: Removed catch block from newBuilder(HttpRequest)
- ... and 5 more: https://git.openjdk.java.net/jdk/compare/5f4e0a40...03b13e7a
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/1059/files
- new: https://git.openjdk.java.net/jdk/pull/1059/files/0de09975..03b13e7a
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1059&range=07
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1059&range=06-07
Stats: 196 lines in 10 files changed: 101 ins; 48 del; 47 mod
Patch: https://git.openjdk.java.net/jdk/pull/1059.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1059/head:pull/1059
PR: https://git.openjdk.java.net/jdk/pull/1059
More information about the net-dev
mailing list