[resteasy-commits] [resteasy/Resteasy] c9175d: [RESTEASY-2700] Initial check-in of reactor-netty ...

Branch: refs/heads/main Home: https://github.com/resteasy/Resteasy Commit: c9175df97c7ccabdcb082582cc9bea127fa5c601 https://github.com/resteasy/Resteasy/commit/c9175df97c7ccabdcb082582cc9be... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngineByteBufReleaseTest.java M resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngineTest.java M resteasy-dependencies-bom/pom.xml M server-adapters/pom.xml A server-adapters/resteasy-reactor-netty/README.md A server-adapters/resteasy-reactor-netty/pom.xml A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ChunkOutputStream.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/Logging.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/NettyUtil.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyContainer.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpRequest.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpResponse.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/BasicTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/ClientBadRequestTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/MonoTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/AsyncIOResource.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/AsyncIOTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/AsyncJaxrsResource.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/AsyncWriter.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/AsyncWriterData.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/BlockingWriter.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/BlockingWriterData.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/Expect100Test.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderTooLongTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/JaxrsAsyncTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/JsonData.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/Netty4ApplicationPathTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/PortAssigningTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/RESTEASY1323Test.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/RESTEASY1325Test.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/ResteasyTrailingSlashTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/SniTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/StreamingOutputTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/SubresourceClassInjectionTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/resteasy_jaxrs/i18n/TestMessagesParent.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/util/SSLCerts.java A server-adapters/resteasy-reactor-netty/src/test/resources/logback.xml Log Message: ----------- [RESTEASY-2700] Initial check-in of reactor-netty based server adapter. Commit: 9d6343634a615b312adc1d1565ecd4e18d48d515 https://github.com/resteasy/Resteasy/commit/9d6343634a615b312adc1d1565ecd... Author: Sharath Srinivasa <sharathsrinivasa(a)users.noreply.github.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/pom.xml M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyContainer.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/BasicResource.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/BasicSSLTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/BasicTest.java A server-adapters/resteasy-reactor-netty/src/test/resources/sni/default_client_truststore.jks A server-adapters/resteasy-reactor-netty/src/test/resources/sni/default_server_keystore.jks A server-adapters/resteasy-reactor-netty/src/test/resources/sni/no_trusted_clients_keystore.jks A server-adapters/resteasy-reactor-netty/src/test/resources/sni/sni_client_truststore.jks A server-adapters/resteasy-reactor-netty/src/test/resources/sni/sni_server_keystore.jks Log Message: ----------- [RESTEASY-2700] Add SSL support for reactor netty server (#2) Commit: 646a5aae0080128a60cf49ef369a2e0ecb934d6a https://github.com/resteasy/Resteasy/commit/646a5aae0080128a60cf49ef369a2... Author: lalifk <lalifk(a)users.noreply.github.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpRequest.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/ClientUncheckedErrorTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UncommonCasesTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java Log Message: ----------- [RESTEASY-2700] - Building UriInfo from full URI & Server exception handling improvements (#3) * Build UriInfo from full URI * Tests to verify UriInfo * Server error handling update * Update server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/ClientUncheckedErrorTest.java Co-authored-by: Sharath Srinivasa <sharathsrinivasa(a)users.noreply.github.com&gt; Co-authored-by: Samuel Cox <crankydillo(a)gmail.com&gt; Co-authored-by: Sharath Srinivasa <sharathsrinivasa(a)users.noreply.github.com&gt; Commit: e15b312f24d8021f15da1f9f7ed8b67a67e1e76d https://github.com/resteasy/Resteasy/commit/e15b312f24d8021f15da1f9f7ed8b... Author: Abhijith Prabhakar <abhi.predator(a)gmail.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/CleanUpTasksTest.java Log Message: ----------- [RESTEASY-2700] Adding clean up tasks to reactor netty JAXRS server Co-authored-by: Abhijith Prabhakar <abprabhakar(a)paypal.com&gt; Commit: 2a4f831bf0d07b91159bde2b26d4dd318680a35f https://github.com/resteasy/Resteasy/commit/2a4f831bf0d07b91159bde2b26d4d... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java Log Message: ----------- [RESTEASY-2700] 'Undo' some unintentional overwrites. Commit: fd69fc225954f93857e20e0a5fcc38263f2a7fd6 https://github.com/resteasy/Resteasy/commit/fd69fc225954f93857e20e0a5fcc3... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ChunkOutputStream.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpRequest.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpResponse.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/i18n/Messages.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderTooLongTest.java A server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeadersLowerCasedTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java Log Message: ----------- [RESTEASY-2700] @anilgursel contributions. (#5) Commit: 7b0ada67c5f8c46b0ebcd961f2c87c8119f933a9 https://github.com/resteasy/Resteasy/commit/7b0ada67c5f8c46b0ebcd961f2c87... Author: lalifk <lalifk(a)users.noreply.github.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ChunkOutputStream.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpResponse.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java A server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/SinkSubscriber.java Log Message: ----------- [RESTEASY-2700] - Removed deprecated processor API code (#6) Commit: 8b292b6f1c16b35211c2b91d01ed357a438b1cc8 https://github.com/resteasy/Resteasy/commit/8b292b6f1c16b35211c2b91d01ed3... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M resteasy-dependencies-bom/pom.xml M server-adapters/resteasy-reactor-netty/pom.xml M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ChunkOutputStream.java R server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/Logging.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/NettyUtil.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyContainer.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpRequest.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyHttpResponse.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/SinkSubscriber.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/i18n/Messages.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/BasicResource.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/BasicTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java R server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/SniTest.java Log Message: ----------- [RESTEASY-2700] Address PR comments. Some notables: use jboss-logging, add HEAD test, deal with several potential NPEs, use JDK `Collections`, and remove SniTest. SNI support can be added later (when someone needs it:) Commit: 1352ea6fda8db59cc7a99ffaf2cdc4e9212409e7 https://github.com/resteasy/Resteasy/commit/1352ea6fda8db59cc7a99ffaf2cdc... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M docbook/reference/en/en-US/modules/RESTEasy_Embedded_Container.xml Log Message: ----------- [RESTEASY-2700] Add reactor-netty entry to embedded server doc. Commit: 10167c10750dccfe688e604892ae69b084fcd3d7 https://github.com/resteasy/Resteasy/commit/10167c10750dccfe688e604892ae6... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderTooLongTest.java Log Message: ----------- [RESTEASY-2700] Work-around a reactor-netty header-too-long problem... [reactor-netty issue](https://github.com/reactor/reactor-netty/issues/1584) which will soon be documented on their codebase. Basically, they fail to read the entire request before closing the connection. This is something we are willing to live with for now.. The test was adjust to show that if the header value is too long (but not too too long:), the server will respond with a 413. While doing this, I noticed that reactor-netty has added its own 'idle timeout' functionality so we'll leverage that instead of registering custom netty handlers. [RESTEASY-2700](https://engineering.paypalcorp.com/jira/browse/RESTEASY-2700) Commit: 5ef07c314246dc3f02d5e3515706caf7c0388d5b https://github.com/resteasy/Resteasy/commit/5ef07c314246dc3f02d5e3515706c... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-04-19 (Mon, 19 Apr 2021) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ChunkOutputStream.java M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java Log Message: ----------- [RESTEASY-2700] Throw NPE at the right point and make a class.. package-private. [RESTEASY-2700](https://engineering.paypalcorp.com/jira/browse/RESTEASY-2700) Compare: https://github.com/resteasy/Resteasy/compare/7875d6e8038d...5ef07c314246