Re: [jfx11u] RFR: 8211308: Support HTTP/2 in WebView
On Thu, 19 Aug 2021 18:39:08 GMT, Johan Vos wrote: > We should probably add this to the release notes. Good idea. - PR: https://git.openjdk.java.net/jfx11u/pull/30
Re: [jfx11u] RFR: 8211308: Support HTTP/2 in WebView
On Tue, 17 Aug 2021 17:13:36 GMT, Kevin Rushforth wrote: > Clean backport of HTTP/2. This is enabled by default only with running with > JDK 12 or later. > > I have tested this with both JDK 11, where I verified that it HTTP/2 is not > enabled, and JDK 15, where I verified that it HTTP/2 is enabled. > > As an FYI, I tested this patch in my > [test-kcr-11.0.13](https://github.com/kevinrushforth/jfx11u/tree/test-kcr-11.0.13) > branch, which has the aggregate set of patches for the following bugs: > [JDK-8211308](https://bugs.openjdk.java.net/browse/JDK-8211308), > [JDK-8268915](https://bugs.openjdk.java.net/browse/JDK-8268915), > [JDK-8269147](https://bugs.openjdk.java.net/browse/JDK-8269147), > [JDK-8269131](https://bugs.openjdk.java.net/browse/JDK-8269131), > [JDK-8268849](https://bugs.openjdk.java.net/browse/JDK-8268849), > [JDK-8270479](https://bugs.openjdk.java.net/browse/JDK-8270479), > [JDK-8272329](https://bugs.openjdk.java.net/browse/JDK-8272329). basic tests work as expected. We should probably add this to the release notes. - Marked as reviewed by jvos (Reviewer). PR: https://git.openjdk.java.net/jfx11u/pull/30
Re: [jfx11u] RFR: 8211308: Support HTTP/2 in WebView
On Thu, 19 Aug 2021 11:42:12 GMT, Kevin Rushforth wrote: > @johanvos Did you want to review this before I integrate it? I'm testing, will have a result in < 4 hours from now - PR: https://git.openjdk.java.net/jfx11u/pull/30
Re: [jfx11u] RFR: 8211308: Support HTTP/2 in WebView
On Tue, 17 Aug 2021 17:13:36 GMT, Kevin Rushforth wrote: > Clean backport of HTTP/2. This is enabled by default only with running with > JDK 12 or later. > > I have tested this with both JDK 11, where I verified that it HTTP/2 is not > enabled, and JDK 15, where I verified that it HTTP/2 is enabled. > > As an FYI, I tested this patch in my > [test-kcr-11.0.13](https://github.com/kevinrushforth/jfx11u/tree/test-kcr-11.0.13) > branch, which has the aggregate set of patches for the following bugs: > [JDK-8211308](https://bugs.openjdk.java.net/browse/JDK-8211308), > [JDK-8268915](https://bugs.openjdk.java.net/browse/JDK-8268915), > [JDK-8269147](https://bugs.openjdk.java.net/browse/JDK-8269147), > [JDK-8269131](https://bugs.openjdk.java.net/browse/JDK-8269131), > [JDK-8268849](https://bugs.openjdk.java.net/browse/JDK-8268849), > [JDK-8270479](https://bugs.openjdk.java.net/browse/JDK-8270479), > [JDK-8272329](https://bugs.openjdk.java.net/browse/JDK-8272329). @johanvos Did you want to review this before I integrate it? - PR: https://git.openjdk.java.net/jfx11u/pull/30
[jfx11u] RFR: 8211308: Support HTTP/2 in WebView
Clean backport of HTTP/2. This is enabled by default only with running with JDK 12 or later. I have tested this with both JDK 11, where I verified that it HTTP/2 is not enabled, and JDK 15, where I verified that it HTTP/2 is enabled. As an FYI, I tested this patch in my [test-kcr-11.0.13](https://github.com/kevinrushforth/jfx11u/tree/test-kcr-11.0.13) branch, which has the aggregate set of patches for the following bugs: [JDK-8211308](https://bugs.openjdk.java.net/browse/JDK-8211308), [JDK-8268915](https://bugs.openjdk.java.net/browse/JDK-8268915), [JDK-8269147](https://bugs.openjdk.java.net/browse/JDK-8269147), [JDK-8269131](https://bugs.openjdk.java.net/browse/JDK-8269131), [JDK-8268849](https://bugs.openjdk.java.net/browse/JDK-8268849), [JDK-8270479](https://bugs.openjdk.java.net/browse/JDK-8270479), [JDK-8272329](https://bugs.openjdk.java.net/browse/JDK-8272329). - Commit messages: - 8211308: Support HTTP/2 in WebView Changes: https://git.openjdk.java.net/jfx11u/pull/30/files Webrev: https://webrevs.openjdk.java.net/?repo=jfx11u=30=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 Stats: 1166 lines in 14 files changed: 881 ins; 217 del; 68 mod Patch: https://git.openjdk.java.net/jfx11u/pull/30.diff Fetch: git fetch https://git.openjdk.java.net/jfx11u pull/30/head:pull/30 PR: https://git.openjdk.java.net/jfx11u/pull/30
Re: [jfx11u] RFR: 8211308: Support HTTP/2 in WebView
On Tue, 17 Aug 2021 17:13:36 GMT, Kevin Rushforth wrote: > Clean backport of HTTP/2. This is enabled by default only with running with > JDK 12 or later. > > I have tested this with both JDK 11, where I verified that it HTTP/2 is not > enabled, and JDK 15, where I verified that it HTTP/2 is enabled. > > As an FYI, I tested this patch in my > [test-kcr-11.0.13](https://github.com/kevinrushforth/jfx11u/tree/test-kcr-11.0.13) > branch, which has the aggregate set of patches for the following bugs: > [JDK-8211308](https://bugs.openjdk.java.net/browse/JDK-8211308), > [JDK-8268915](https://bugs.openjdk.java.net/browse/JDK-8268915), > [JDK-8269147](https://bugs.openjdk.java.net/browse/JDK-8269147), > [JDK-8269131](https://bugs.openjdk.java.net/browse/JDK-8269131), > [JDK-8268849](https://bugs.openjdk.java.net/browse/JDK-8268849), > [JDK-8270479](https://bugs.openjdk.java.net/browse/JDK-8270479), > [JDK-8272329](https://bugs.openjdk.java.net/browse/JDK-8272329). @johanvos Even though this is a clean backport, I'd like you to review it before I integrate it. - PR: https://git.openjdk.java.net/jfx11u/pull/30
Re: [Integrated] RFR: 8211308: Support HTTP/2 in WebView
Changeset: 98035cb2 Author:Arunprasad Rajkumar Date: 2019-12-03 08:24:01 + URL: https://git.openjdk.java.net/jfx/commit/98035cb2 8211308: Support HTTP/2 in WebView Reviewed-by: ghb, kcr ! modules/javafx.web/src/main/java/com/sun/webkit/network/ByteBufferPool.java + modules/javafx.web/src/main/java/com/sun/webkit/network/HTTP2Loader.java ! modules/javafx.web/src/main/java/com/sun/webkit/network/NetworkContext.java ! modules/javafx.web/src/main/java/com/sun/webkit/network/URLLoader.java + modules/javafx.web/src/main/java/com/sun/webkit/network/URLLoaderBase.java ! modules/javafx.web/src/main/java/module-info.java ! modules/javafx.web/src/main/native/Source/WebCore/mapfile-macosx ! modules/javafx.web/src/main/native/Source/WebCore/mapfile-vers ! modules/javafx.web/src/main/native/Source/WebCore/platform/java/IDNJava.cpp ! modules/javafx.web/src/main/native/Source/WebCore/platform/network/ResourceHandle.h ! modules/javafx.web/src/main/native/Source/WebCore/platform/network/ResourceHandleInternal.h ! modules/javafx.web/src/main/native/Source/WebCore/platform/network/java/ResourceHandleJava.cpp ! modules/javafx.web/src/main/native/Source/WebCore/platform/network/java/URLLoader.cpp ! modules/javafx.web/src/main/native/Source/WebCore/platform/network/java/URLLoader.h
Re: [Approved] RFR: 8211308: Support HTTP/2 in WebView
On Wed, 16 Oct 2019 17:57:58 GMT, Arunprasad Rajkumar wrote: > The pull request has been updated with additional changes. > > > > Added commits: > - 1832c2db: Incorporate fixes provided by @kcr > > Changes: > - all: https://git.openjdk.java.net/jfx/pull/14/files > - new: https://git.openjdk.java.net/jfx/pull/14/files/1798a661..1832c2db > > Webrevs: > - full: https://webrevs.openjdk.java.net/jfx/14/webrev.01 > - incr: https://webrevs.openjdk.java.net/jfx/14/webrev.00-01 > > Stats: 13 lines in 2 files changed: 5 ins; 0 del; 8 mod > Patch: https://git.openjdk.java.net/jfx/pull/14.diff > Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 Approved by kcr (Lead). PR: https://git.openjdk.java.net/jfx/pull/14
Re: [Approved] RFR: 8211308: Support HTTP/2 in WebView
On Wed, 16 Oct 2019 17:57:58 GMT, Arunprasad Rajkumar wrote: > The pull request has been updated with additional changes. > > > > Added commits: > - 1832c2db: Incorporate fixes provided by @kcr > > Changes: > - all: https://git.openjdk.java.net/jfx/pull/14/files > - new: https://git.openjdk.java.net/jfx/pull/14/files/1798a661..1832c2db > > Webrevs: > - full: https://webrevs.openjdk.java.net/jfx/14/webrev.01 > - incr: https://webrevs.openjdk.java.net/jfx/14/webrev.00-01 > > Stats: 13 lines in 2 files changed: 5 ins; 0 del; 8 mod > Patch: https://git.openjdk.java.net/jfx/pull/14.diff > Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 Looks good to me. Approved by ghb (Reviewer). PR: https://git.openjdk.java.net/jfx/pull/14
Re: RFR: 8211308: Support HTTP/2 in WebView
On Wed, 16 Oct 2019 21:43:15 GMT, Kevin Rushforth wrote: > On Wed, 16 Oct 2019 18:10:00 GMT, Kevin Rushforth wrote: > >> On Wed, 16 Oct 2019 17:58:32 GMT, Arunprasad Rajkumar >> wrote: >> >>> On Fri, 11 Oct 2019 11:21:08 GMT, Robin Westberg >>> wrote: >>> On Fri, 11 Oct 2019 07:01:48 GMT, Arunprasad Rajkumar wrote: > On Fri, 11 Oct 2019 06:44:09 GMT, Johan Vos wrote: > >> On Fri, 11 Oct 2019 06:18:38 GMT, Arunprasad Rajkumar >> wrote: >> >>> On Fri, 11 Oct 2019 06:07:14 GMT, Arunprasad Rajkumar >>> wrote: >>> The goal of this enhancement is to use new [HttpClient APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) available from JDK 11. Reference: [1] https://openjdk.java.net/groups/net/httpclient/intro.html [2] https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work correctly due to the dependency on following issues, [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable to connect to https://google.com using java.net.HttpClient [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow 204 responses with Content-Length:0 [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) java.net.http HTTP client should allow specifying Origin and Referer headers Task List - [x] simple GET requests - [x] Runtime setting to fallback to legacy client - [ ] Runtime settings to use *only* HTTP/1.1 - [x] sync requests - [x] Error Handling & Propagation - [x] POST with form data - [x] AccessController association for HttpClient.sendAsync / send - [x] Redirection - [ ] Check for possibilities to write unit tests - [ ] Sync request handling from WebCore java platform layer - [x] Make use of singleton instance of direct ByteBuffer instead of using allocator pool - [x] gzip, deflate encoding support HTTP/2 Test pages - http://www.http2demo.io - https://http2.akamai.com/demo - https://http2.golang.org - https://google.com Redirection Test - https://www.httpwatch.com/httpgallery/redirection/#showExample7 More details are available at https://github.com/javafxports/openjdk-jfx/pull/247. Commits: - 1798a661: 8211308: Support HTTP/2 in WebView Changes: https://git.openjdk.java.net/jfx/pull/14/files Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod Patch: https://git.openjdk.java.net/jfx/pull/14.diff Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 >>> >>> Still few changes need to be done as [suggested >>> by](https://github.com/javafxports/openjdk-jfx/pull/247#pullrequestreview-283699613) >>> @kevinrushforth. >> >> Good work. Should the title be prefixed with WIP until it's ready for >> review, so that Skara will send the RFR when it is ready for review? > > I was wondering why @skara had sent the RFR when the PR is still in draft > stage. Actually @skara should consider the "Draft" attribute associated > with the PR. Good point, I've created https://bugs.openjdk.java.net/browse/SKARA-129 to track this. >>> >>> @jfx team, now it is ready for a fresh review :) >> >> @guruhb please also review this. > > I note that this change needs two reviewers (so should not be integrated > until there are two approved reviews). I did a fair bit of testing yesterday on all three platforms, and it all looks good to me. I verified the behavior on JDK 11 LTS (not enabled by default) and on JDK 13 (is enabled by default). I have one more things to check next week. @guruhb can you also review? PR: https://git.openjdk.java.net/jfx/pull/14
Re: RFR: 8211308: Support HTTP/2 in WebView
On Wed, 16 Oct 2019 18:10:00 GMT, Kevin Rushforth wrote: > On Wed, 16 Oct 2019 17:58:32 GMT, Arunprasad Rajkumar > wrote: > >> On Fri, 11 Oct 2019 11:21:08 GMT, Robin Westberg >> wrote: >> >>> On Fri, 11 Oct 2019 07:01:48 GMT, Arunprasad Rajkumar >>> wrote: >>> On Fri, 11 Oct 2019 06:44:09 GMT, Johan Vos wrote: > On Fri, 11 Oct 2019 06:18:38 GMT, Arunprasad Rajkumar > wrote: > >> On Fri, 11 Oct 2019 06:07:14 GMT, Arunprasad Rajkumar >> wrote: >> >>> The goal of this enhancement is to use new [HttpClient >>> APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) >>> available from JDK 11. >>> >>> Reference: >>> [1] https://openjdk.java.net/groups/net/httpclient/intro.html >>> [2] >>> https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html >>> >>> Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work >>> correctly due to the dependency on following issues, >>> >>> [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable >>> to connect to https://google.com using java.net.HttpClient >>> [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow >>> 204 responses with Content-Length:0 >>> [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) >>> java.net.http HTTP client should allow specifying Origin and Referer >>> headers >>> >>> Task List >>> - [x] simple GET requests >>> - [x] Runtime setting to fallback to legacy client >>> - [ ] Runtime settings to use *only* HTTP/1.1 >>> - [x] sync requests >>> - [x] Error Handling & Propagation >>> - [x] POST with form data >>> - [x] AccessController association for HttpClient.sendAsync / send >>> - [x] Redirection >>> - [ ] Check for possibilities to write unit tests >>> - [ ] Sync request handling from WebCore java platform layer >>> - [x] Make use of singleton instance of direct ByteBuffer instead of >>> using allocator pool >>> - [x] gzip, deflate encoding support >>> >>> HTTP/2 Test pages >>> - http://www.http2demo.io >>> - https://http2.akamai.com/demo >>> - https://http2.golang.org >>> - https://google.com >>> >>> Redirection Test >>> - https://www.httpwatch.com/httpgallery/redirection/#showExample7 >>> >>> More details are available at >>> https://github.com/javafxports/openjdk-jfx/pull/247. >>> >>> >>> >>> Commits: >>> - 1798a661: 8211308: Support HTTP/2 in WebView >>> >>> Changes: https://git.openjdk.java.net/jfx/pull/14/files >>> Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 >>> Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 >>> Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod >>> Patch: https://git.openjdk.java.net/jfx/pull/14.diff >>> Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 >> >> Still few changes need to be done as [suggested >> by](https://github.com/javafxports/openjdk-jfx/pull/247#pullrequestreview-283699613) >> @kevinrushforth. > > Good work. Should the title be prefixed with WIP until it's ready for > review, so that Skara will send the RFR when it is ready for review? I was wondering why @skara had sent the RFR when the PR is still in draft stage. Actually @skara should consider the "Draft" attribute associated with the PR. >>> >>> Good point, I've created https://bugs.openjdk.java.net/browse/SKARA-129 to >>> track this. >> >> @jfx team, now it is ready for a fresh review :) > > @guruhb please also review this. I note that this change needs two reviewers (so should not be integrated until there are two approved reviews). PR: https://git.openjdk.java.net/jfx/pull/14
Re: RFR: 8211308: Support HTTP/2 in WebView
On Wed, 16 Oct 2019 17:58:32 GMT, Arunprasad Rajkumar wrote: > On Fri, 11 Oct 2019 11:21:08 GMT, Robin Westberg > wrote: > >> On Fri, 11 Oct 2019 07:01:48 GMT, Arunprasad Rajkumar >> wrote: >> >>> On Fri, 11 Oct 2019 06:44:09 GMT, Johan Vos wrote: >>> On Fri, 11 Oct 2019 06:18:38 GMT, Arunprasad Rajkumar wrote: > On Fri, 11 Oct 2019 06:07:14 GMT, Arunprasad Rajkumar > wrote: > >> The goal of this enhancement is to use new [HttpClient >> APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) >> available from JDK 11. >> >> Reference: >> [1] https://openjdk.java.net/groups/net/httpclient/intro.html >> [2] >> https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html >> >> Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work >> correctly due to the dependency on following issues, >> >> [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable >> to connect to https://google.com using java.net.HttpClient >> [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow >> 204 responses with Content-Length:0 >> [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) >> java.net.http HTTP client should allow specifying Origin and Referer >> headers >> >> Task List >> - [x] simple GET requests >> - [x] Runtime setting to fallback to legacy client >> - [ ] Runtime settings to use *only* HTTP/1.1 >> - [x] sync requests >> - [x] Error Handling & Propagation >> - [x] POST with form data >> - [x] AccessController association for HttpClient.sendAsync / send >> - [x] Redirection >> - [ ] Check for possibilities to write unit tests >> - [ ] Sync request handling from WebCore java platform layer >> - [x] Make use of singleton instance of direct ByteBuffer instead of >> using allocator pool >> - [x] gzip, deflate encoding support >> >> HTTP/2 Test pages >> - http://www.http2demo.io >> - https://http2.akamai.com/demo >> - https://http2.golang.org >> - https://google.com >> >> Redirection Test >> - https://www.httpwatch.com/httpgallery/redirection/#showExample7 >> >> More details are available at >> https://github.com/javafxports/openjdk-jfx/pull/247. >> >> >> >> Commits: >> - 1798a661: 8211308: Support HTTP/2 in WebView >> >> Changes: https://git.openjdk.java.net/jfx/pull/14/files >> Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 >> Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 >> Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod >> Patch: https://git.openjdk.java.net/jfx/pull/14.diff >> Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 > > Still few changes need to be done as [suggested > by](https://github.com/javafxports/openjdk-jfx/pull/247#pullrequestreview-283699613) > @kevinrushforth. Good work. Should the title be prefixed with WIP until it's ready for review, so that Skara will send the RFR when it is ready for review? >>> >>> I was wondering why @skara had sent the RFR when the PR is still in draft >>> stage. Actually @skara should consider the "Draft" attribute associated >>> with the PR. >> >> Good point, I've created https://bugs.openjdk.java.net/browse/SKARA-129 to >> track this. > > @jfx team, now it is ready for a fresh review :) @guruhb please also review this. PR: https://git.openjdk.java.net/jfx/pull/14
Re: RFR: 8211308: Support HTTP/2 in WebView
On Fri, 11 Oct 2019 11:21:08 GMT, Robin Westberg wrote: > On Fri, 11 Oct 2019 07:01:48 GMT, Arunprasad Rajkumar > wrote: > >> On Fri, 11 Oct 2019 06:44:09 GMT, Johan Vos wrote: >> >>> On Fri, 11 Oct 2019 06:18:38 GMT, Arunprasad Rajkumar >>> wrote: >>> On Fri, 11 Oct 2019 06:07:14 GMT, Arunprasad Rajkumar wrote: > The goal of this enhancement is to use new [HttpClient > APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) > available from JDK 11. > > Reference: > [1] https://openjdk.java.net/groups/net/httpclient/intro.html > [2] > https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html > > Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work > correctly due to the dependency on following issues, > > [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable to > connect to https://google.com using java.net.HttpClient > [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow 204 > responses with Content-Length:0 > [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) > java.net.http HTTP client should allow specifying Origin and Referer > headers > > Task List > - [x] simple GET requests > - [x] Runtime setting to fallback to legacy client > - [ ] Runtime settings to use *only* HTTP/1.1 > - [x] sync requests > - [x] Error Handling & Propagation > - [x] POST with form data > - [x] AccessController association for HttpClient.sendAsync / send > - [x] Redirection > - [ ] Check for possibilities to write unit tests > - [ ] Sync request handling from WebCore java platform layer > - [x] Make use of singleton instance of direct ByteBuffer instead of > using allocator pool > - [x] gzip, deflate encoding support > > HTTP/2 Test pages > - http://www.http2demo.io > - https://http2.akamai.com/demo > - https://http2.golang.org > - https://google.com > > Redirection Test > - https://www.httpwatch.com/httpgallery/redirection/#showExample7 > > More details are available at > https://github.com/javafxports/openjdk-jfx/pull/247. > > > > Commits: > - 1798a661: 8211308: Support HTTP/2 in WebView > > Changes: https://git.openjdk.java.net/jfx/pull/14/files > Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 > Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 > Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod > Patch: https://git.openjdk.java.net/jfx/pull/14.diff > Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 Still few changes need to be done as [suggested by](https://github.com/javafxports/openjdk-jfx/pull/247#pullrequestreview-283699613) @kevinrushforth. >>> >>> Good work. Should the title be prefixed with WIP until it's ready for >>> review, so that Skara will send the RFR when it is ready for review? >> >> I was wondering why @skara had sent the RFR when the PR is still in draft >> stage. Actually @skara should consider the "Draft" attribute associated with >> the PR. > > Good point, I've created https://bugs.openjdk.java.net/browse/SKARA-129 to > track this. @jfx team, now it is ready for a fresh review :) PR: https://git.openjdk.java.net/jfx/pull/14
Re: RFR: 8211308: Support HTTP/2 in WebView
On Fri, 11 Oct 2019 06:18:38 GMT, Arunprasad Rajkumar wrote: > On Fri, 11 Oct 2019 06:07:14 GMT, Arunprasad Rajkumar > wrote: > >> The goal of this enhancement is to use new [HttpClient >> APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) >> available from JDK 11. >> >> Reference: >> [1] https://openjdk.java.net/groups/net/httpclient/intro.html >> [2] >> https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html >> >> Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work >> correctly due to the dependency on following issues, >> >> [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable to >> connect to https://google.com using java.net.HttpClient >> [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow 204 >> responses with Content-Length:0 >> [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) >> java.net.http HTTP client should allow specifying Origin and Referer headers >> >> Task List >> - [x] simple GET requests >> - [x] Runtime setting to fallback to legacy client >> - [ ] Runtime settings to use *only* HTTP/1.1 >> - [x] sync requests >> - [x] Error Handling & Propagation >> - [x] POST with form data >> - [x] AccessController association for HttpClient.sendAsync / send >> - [x] Redirection >> - [ ] Check for possibilities to write unit tests >> - [ ] Sync request handling from WebCore java platform layer >> - [x] Make use of singleton instance of direct ByteBuffer instead of using >> allocator pool >> - [x] gzip, deflate encoding support >> >> HTTP/2 Test pages >> - http://www.http2demo.io >> - https://http2.akamai.com/demo >> - https://http2.golang.org >> - https://google.com >> >> Redirection Test >> - https://www.httpwatch.com/httpgallery/redirection/#showExample7 >> >> More details are available at >> https://github.com/javafxports/openjdk-jfx/pull/247. >> >> >> >> Commits: >> - 1798a661: 8211308: Support HTTP/2 in WebView >> >> Changes: https://git.openjdk.java.net/jfx/pull/14/files >> Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 >> Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 >> Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod >> Patch: https://git.openjdk.java.net/jfx/pull/14.diff >> Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 > > Still few changes need to be done as [suggested > by](https://github.com/javafxports/openjdk-jfx/pull/247#pullrequestreview-283699613) > @kevinrushforth. Good work. Should the title be prefixed with WIP until it's ready for review, so that Skara will send the RFR when it is ready for review? PR: https://git.openjdk.java.net/jfx/pull/14
Re: RFR: 8211308: Support HTTP/2 in WebView
On Fri, 11 Oct 2019 06:07:14 GMT, Arunprasad Rajkumar wrote: > The goal of this enhancement is to use new [HttpClient > APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) > available from JDK 11. > > Reference: > [1] https://openjdk.java.net/groups/net/httpclient/intro.html > [2] > https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html > > Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work > correctly due to the dependency on following issues, > > [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable to > connect to https://google.com using java.net.HttpClient > [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow 204 > responses with Content-Length:0 > [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) java.net.http > HTTP client should allow specifying Origin and Referer headers > > Task List > - [x] simple GET requests > - [x] Runtime setting to fallback to legacy client > - [ ] Runtime settings to use *only* HTTP/1.1 > - [x] sync requests > - [x] Error Handling & Propagation > - [x] POST with form data > - [x] AccessController association for HttpClient.sendAsync / send > - [x] Redirection > - [ ] Check for possibilities to write unit tests > - [ ] Sync request handling from WebCore java platform layer > - [x] Make use of singleton instance of direct ByteBuffer instead of using > allocator pool > - [x] gzip, deflate encoding support > > HTTP/2 Test pages > - http://www.http2demo.io > - https://http2.akamai.com/demo > - https://http2.golang.org > - https://google.com > > Redirection Test > - https://www.httpwatch.com/httpgallery/redirection/#showExample7 > > More details are available at > https://github.com/javafxports/openjdk-jfx/pull/247. > > > > Commits: > - 1798a661: 8211308: Support HTTP/2 in WebView > > Changes: https://git.openjdk.java.net/jfx/pull/14/files > Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 > Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 > Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod > Patch: https://git.openjdk.java.net/jfx/pull/14.diff > Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 Still few changes need to be done as [suggested by](https://github.com/javafxports/openjdk-jfx/pull/247#pullrequestreview-283699613) @kevinrushforth. PR: https://git.openjdk.java.net/jfx/pull/14
RFR: 8211308: Support HTTP/2 in WebView
The goal of this enhancement is to use new [HttpClient APIs](https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html) available from JDK 11. Reference: [1] https://openjdk.java.net/groups/net/httpclient/intro.html [2] https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpClient.html Though this uses JDK 11 HttpClient APIs, it needs latest JDK 12 to work correctly due to the dependency on following issues, [JDK-8218546](https://bugs.openjdk.java.net/browse/JDK-8218546) Unable to connect to https://google.com using java.net.HttpClient [JDK-8218662](https://bugs.openjdk.java.net/browse/JDK-8218662) Allow 204 responses with Content-Length:0 [JDK-8203850](https://bugs.openjdk.java.net/browse/JDK-8203850) java.net.http HTTP client should allow specifying Origin and Referer headers Task List - [x] simple GET requests - [x] Runtime setting to fallback to legacy client - [ ] Runtime settings to use *only* HTTP/1.1 - [x] sync requests - [x] Error Handling & Propagation - [x] POST with form data - [x] AccessController association for HttpClient.sendAsync / send - [x] Redirection - [ ] Check for possibilities to write unit tests - [ ] Sync request handling from WebCore java platform layer - [x] Make use of singleton instance of direct ByteBuffer instead of using allocator pool - [x] gzip, deflate encoding support HTTP/2 Test pages - http://www.http2demo.io - https://http2.akamai.com/demo - https://http2.golang.org - https://google.com Redirection Test - https://www.httpwatch.com/httpgallery/redirection/#showExample7 More details are available at https://github.com/javafxports/openjdk-jfx/pull/247. Commits: - 1798a661: 8211308: Support HTTP/2 in WebView Changes: https://git.openjdk.java.net/jfx/pull/14/files Webrev: https://webrevs.openjdk.java.net/jfx/14/webrev.00 Issue: https://bugs.openjdk.java.net/browse/JDK-8211308 Stats: 1161 lines in 14 files changed: 876 ins; 217 del; 68 mod Patch: https://git.openjdk.java.net/jfx/pull/14.diff Fetch: git fetch https://git.openjdk.java.net/jfx pull/14/head:pull/14 PR: https://git.openjdk.java.net/jfx/pull/14
[RFR] 8211308: Support HTTP/2 in WebView
Hi, Please review the following Github PR which uses new HttpClient interface (introduced since JDK11[1]) to support HTTP/2. https://github.com/javafxports/openjdk-jfx/pull/247 https://bugs.openjdk.java.net/browse/JDK-8211308 Note: There is a runtime property “com.sun.webkit.useHTTP2Loader" added to switch back to legacy interface. By default it is enabled to use new HttpClient interface. [1] https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/package-summary.html Thanks, Arun