Re: RFR: JDK-8061729 : Update java/net tests to eliminate dependency on sun.net.www.MessageHeader and some other internal APIs [v5]
> There are some regression tests depending on sun.net.www.MessageHeader, the > internal API dependency should be removed. Some of other internal API > dependancies are removed in following issues : > JDK-8273142 > JDK-8268464 > JDK-8268133 Mahendra Chhipa has updated the pull request incrementally with one additional commit since the last revision: 1. Handled continuation line 2. handled \r, \n, or \r\n 3. Added the test for HttpHeaderParser class - Changes: - all: https://git.openjdk.java.net/jdk/pull/5937/files - new: https://git.openjdk.java.net/jdk/pull/5937/files/6f4b7d3a..70a37be9 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5937&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5937&range=03-04 Stats: 857 lines in 11 files changed: 812 ins; 10 del; 35 mod Patch: https://git.openjdk.java.net/jdk/pull/5937.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5937/head:pull/5937 PR: https://git.openjdk.java.net/jdk/pull/5937
Re: RFR: 8278067: Make HttpURLConnection default keep alive timeout configurable [v2]
> Hi, > > Could I get the following patch reviewed please? (A CSR is also required > which I will submit when the docs are agreed) > > It adds a pair of new system properties to make the keep alive timer in > java.net.HttpURLConnection configurable. > The proposed property names are: > > "http.keepAlive.time.server" and "http.keepAlive.time.proxy" > > Thanks, > Michael Michael McMahon 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 11 additional commits since the last revision: - update after Daniel's first review - Merge branch 'master' into keepalive - added docs - reverted change to LIFETIME constant. Were not necessary - updates - Merge branch 'master' into keepalive - Merge branch 'master' into keepalive - Merge branch 'master' into keepalive - Merge branch 'master' into keepalive - Merge branch 'master' into keepalive - ... and 1 more: https://git.openjdk.java.net/jdk/compare/98c4ce4c...edcbb13f - Changes: - all: https://git.openjdk.java.net/jdk/pull/7349/files - new: https://git.openjdk.java.net/jdk/pull/7349/files/85d5662b..edcbb13f Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7349&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7349&range=00-01 Stats: 12764 lines in 314 files changed: 9568 ins; 1658 del; 1538 mod Patch: https://git.openjdk.java.net/jdk/pull/7349.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7349/head:pull/7349 PR: https://git.openjdk.java.net/jdk/pull/7349
