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

Branch: refs/heads/4.7 Home: https://github.com/resteasy/resteasy Commit: 6a2e6848eaf1b33e398ab097fc00ee4ea9a007aa https://github.com/resteasy/resteasy/commit/6a2e6848eaf1b33e398ab097fc00e... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-15 (Wed, 15 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: 54505e64c174b3761081c5358f5ab8e57860c856 https://github.com/resteasy/resteasy/commit/54505e64c174b3761081c5358f5ab... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-21 (Tue, 21 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: 7bac4f40a6532db7904abacfcab8c4d27c4d8f8f https://github.com/resteasy/resteasy/commit/7bac4f40a6532db7904abacfcab8c... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-21 (Tue, 21 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: 20da4556c592d09f4a11a53bd14194a4c4e8d013 https://github.com/resteasy/resteasy/commit/20da4556c592d09f4a11a53bd1419... Author: dnagarajan89 <dnagarajan89(a)gmail.com&gt; Date: 2021-09-21 (Tue, 21 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: 69a0003943c6cd846ee149b895bf5206d6956dbb https://github.com/resteasy/resteasy/commit/69a0003943c6cd846ee149b895bf5... Author: dnagarajan89 <dnagarajan89(a)gmail.com&gt; Date: 2021-09-22 (Wed, 22 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: 0afd328c5dbb135c1fa7e1059b5c35e12f9ad7b0 https://github.com/resteasy/resteasy/commit/0afd328c5dbb135c1fa7e1059b5c3... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2021-09-22 (Wed, 22 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: a8e8042ea3b16096163141d1212ff229baf75a6b https://github.com/resteasy/resteasy/commit/a8e8042ea3b16096163141d1212ff... 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 #2902 from crankydillo/reactor-netty-client-provider-fix Avoid a payload byte[] copy when using reactor-netty HTTP client engine Compare: https://github.com/resteasy/resteasy/compare/a1adb8aa1f08...a8e8042ea3b1