[resteasy-commits] [resteasy/resteasy] deb4cd: 3 additions, all targeting the reactor-netty clien...

Branch: refs/heads/main Home: https://github.com/resteasy/resteasy Commit: deb4cd4f0c49047bf1ddf7d688b41b1ab8ababf0 https://github.com/resteasy/resteasy/commit/deb4cd4f0c49047bf1ddf7d688b41... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngine.java M resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngineTest.java A resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/internal/TrackingClientRequestHeaders.java A resteasy-core/src/main/java/org/jboss/resteasy/util/TrackingMap.java M testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/CaseInsentiveMapTest.java A testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/TrackingMapTest.java Log Message: ----------- 3 additions, all targeting the reactor-netty client engine: 1. Some memory copying is avoided by directly connecting resteasy's request payload to the reactor-netty's 'outbound'. 2. Fix a bug where headers added by things like RestEasy MessageBodyWriters are appropriately added to the request that reactor-netty sends out. 3. If empty due to thread switching, establish ResteasyContext that has the Provider->ClientConfiguration such that Provider field level injection will work in when the request body is written. Much credit for these changes is owed to @anilgursel and @dnagarajan89. Commit: 3b8aa53b2966224a433af135cc3533aff71d112e https://github.com/resteasy/resteasy/commit/3b8aa53b2966224a433af135cc353... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/CaseInsentiveMapTest.java M testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/TrackingMapTest.java Log Message: ----------- Actually use the derived case insensitive map in the base tests. And switch to a non-mutating injection technique. Commit: 5131cfbcf5a6176ae4e0a4d4b95fee36e2fdc524 https://github.com/resteasy/resteasy/commit/5131cfbcf5a6176ae4e0a4d4b95fe... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngine.java Log Message: ----------- Remove the unused method that was copying bytes.. Avoiding this was the main reason we started this work. Commit: 6dc88f43ef4493630c874d777f36469f70ff799b https://github.com/resteasy/resteasy/commit/6dc88f43ef4493630c874d777f364... Author: dnagarajan89 <dnagarajan89(a)gmail.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngineTest.java Log Message: ----------- Fixed PR comments (#8) Co-authored-by: dhanagarajan <dhanagarajan(a)paypal.com&gt; Commit: 6a0ad314df68ccc5d1d6ef0324465cb2282809e3 https://github.com/resteasy/resteasy/commit/6a0ad314df68ccc5d1d6ef0324465... Author: dnagarajan89 <dnagarajan89(a)gmail.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngine.java A resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/internal/TrackingClientRequestHeaders.java A resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/internal/TrackingMap.java A resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/internal/TrackingMapTest.java R resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/internal/TrackingClientRequestHeaders.java R resteasy-core/src/main/java/org/jboss/resteasy/util/TrackingMap.java M testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/CaseInsentiveMapTest.java R testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/TrackingMapTest.java Log Message: ----------- Moving TrackingMap and TrackingClientRequestHeaders to resteasy-client-reactor-netty module. (#9) * Fixed PR comments * Moved TrackingMap and TrackingClientRequestHeaders to resteasy-client-reactor-netty module. Co-authored-by: dhanagarajan <dhanagarajan(a)paypal.com&gt; Commit: 3eea097ea48db5ac1520ce78f7f308581d6c433a https://github.com/resteasy/resteasy/commit/3eea097ea48db5ac1520ce78f7f30... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/CaseInsentiveMapTest.java Log Message: ----------- Remove unused extension point. Commit: 58ce03106522e081c926c620ebbc31a9466c40f8 https://github.com/resteasy/resteasy/commit/58ce03106522e081c926c620ebbc3... Author: James R. Perkins <jperkins(a)redhat.com&gt; Date: 2021-09-23 (Thu, 23 Sep 2021) Changed paths: M resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngine.java A resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/internal/TrackingClientRequestHeaders.java A resteasy-client-reactor-netty/src/main/java/org/jboss/resteasy/client/jaxrs/internal/TrackingMap.java M resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/engines/ReactorNettyClientHttpEngineTest.java A resteasy-client-reactor-netty/src/test/java/org/jboss/resteasy/client/jaxrs/internal/TrackingMapTest.java M testsuite/unit-tests/src/test/java/org/jboss/resteasy/test/util/CaseInsentiveMapTest.java Log Message: ----------- Merge pull request #2910 from jamezp/RESTEASY-3015 [RESTEASY-3015] Avoid a payload byte[] copy when using reactor-netty HTTP client engine Compare: https://github.com/resteasy/resteasy/compare/33f30e9f0025...58ce03106522