[resteasy/resteasy] 21e93c: [RESTEASY-3260] Add a custom constructor for the Y...
by James R. Perkins
Branch: refs/heads/3.15
Home: https://github.com/resteasy/resteasy
Commit: 21e93c195e91400add5208120669819fe4c87dcb
https://github.com/resteasy/resteasy/commit/21e93c195e91400add52081206698...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-10 (Sat, 10 Dec 2022)
Changed paths:
M providers/yaml/src/main/java/org/jboss/resteasy/plugins/providers/YamlProvider.java
M providers/yaml/src/main/java/org/jboss/resteasy/plugins/providers/yaml/i18n/LogMessages.java
M providers/yaml/src/main/java/org/jboss/resteasy/plugins/providers/yaml/i18n/Messages.java
M resteasy-dependencies-bom/pom.xml
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/BadActorYamlProviderTest.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/resource/AttackVector.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/resource/Message.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/resource/MessageResource.java
Log Message:
-----------
[RESTEASY-3260] Add a custom constructor for the YAML provider which only allows specific types.
https://issues.redhat.com/browse/RESTEASY-3260
Signed-off-by: James R. Perkins <jperkins(a)redhat.com>
Commit: ef9f6049f6d00527c6f1cd5d3e603afa1af66c23
https://github.com/resteasy/resteasy/commit/ef9f6049f6d00527c6f1cd5d3e603...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-13 (Tue, 13 Dec 2022)
Changed paths:
M providers/yaml/src/main/java/org/jboss/resteasy/plugins/providers/YamlProvider.java
M providers/yaml/src/main/java/org/jboss/resteasy/plugins/providers/yaml/i18n/LogMessages.java
M providers/yaml/src/main/java/org/jboss/resteasy/plugins/providers/yaml/i18n/Messages.java
M resteasy-dependencies-bom/pom.xml
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/BadActorYamlProviderTest.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/resource/AttackVector.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/resource/Message.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/providers/yaml/resource/MessageResource.java
Log Message:
-----------
Merge pull request #3352 from jamezp/RESTEASY-3260
[RESTEASY-3260] Add a custom constructor for the YAML provider which …
Compare: https://github.com/resteasy/resteasy/compare/2f09a18180c1...ef9f6049f6d0
2 years
[resteasy/resteasy] a77b9f: Use getHostName over getHostString when setting RE...
by James R. Perkins
Branch: refs/heads/5.0
Home: https://github.com/resteasy/resteasy
Commit: a77b9f2dacb1d30dae3a50d6a5a6eb7b1d06069d
https://github.com/resteasy/resteasy/commit/a77b9f2dacb1d30dae3a50d6a5a6e...
Author: Samuel Cox <sacox(a)paypal.com>
Date: 2022-10-07 (Fri, 07 Oct 2022)
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/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
Use getHostName over getHostString when setting RE's URI object.
We want to make a greater effort at capturing the actual hostname used
by the client.
Commit: 3d5c9f2d487a238f519629fc574a00208294a9a9
https://github.com/resteasy/resteasy/commit/3d5c9f2d487a238f519629fc574a0...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-10-11 (Tue, 11 Oct 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java
Log Message:
-----------
Remove a stale comment.
Commit: 48db1a291a719f2554d965663734b0d02be471de
https://github.com/resteasy/resteasy/commit/48db1a291a719f2554d965663734b...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-10-11 (Tue, 11 Oct 2022)
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/plugins/server/reactor/netty/UriInfoTest.java
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java
Log Message:
-----------
The request URI should be governed by the client,
which means we need to read it from the Host header. Not sure why
we didn't just copy what the other adapters did. Honestly, some of
that should probably be abstracted _if_ reading from Host header is
truly the right thing to do. I _think_ we shied away from this because
it felt wrong to return 'unknown'. While that still feels wrong, I'm
not sure what is a better option, so copying. I briefly thought about
returning the IP address, which is what was happening before this effort
to _not_ return the IP address!
Commit: f61707e55577a1695b9908ecdae1902f7f54411f
https://github.com/resteasy/resteasy/commit/f61707e55577a1695b9908ecdae19...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-01 (Thu, 01 Dec 2022)
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/plugins/server/reactor/netty/UriInfoTest.java
R server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java
Log Message:
-----------
Use the IP address if the Host header is invalid.
Commit: 31913ed34c99d7d339f74d8c7dc426d6841d2222
https://github.com/resteasy/resteasy/commit/31913ed34c99d7d339f74d8c7dc42...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-03 (Sat, 03 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/pom.xml
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/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
Throw IllegalArgumentException if the host address cannot be extracted..
from the request. We discussed 'asserting' here. Maybe that is the
best way. I just haven't seen anyone do that in quite a while.
Commit: 4ee135b2d665e2154b86771130ed159f58b65fce
https://github.com/resteasy/resteasy/commit/4ee135b2d665e2154b86771130ed1...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-09 (Fri, 09 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
Remove an empty comment.
Commit: 417393f78f1b4b28da0de104aea38df60498da94
https://github.com/resteasy/resteasy/commit/417393f78f1b4b28da0de104aea38...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
[RESTEASY-3261] Fix the UriInfoTest.
https://issues.redhat.com/browse/RESTEASY-3261
Signed-off-by: James R. Perkins <jperkins(a)redhat.com>
Commit: f31351a63187989ad14adc62f8980988084d7f0b
https://github.com/resteasy/resteasy/commit/f31351a63187989ad14adc62f8980...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/pom.xml
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/plugins/server/reactor/netty/UriInfoTest.java
R 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:
-----------
Merge pull request #3361 from jamezp/reactor-netty-server-uriinfo
Use the request's Host header when setting RE's URI object.
Compare: https://github.com/resteasy/resteasy/compare/6149edbe0ce2...f31351a63187
2 years
[resteasy/resteasy] cdf0c1: Use getHostName over getHostString when setting RE...
by James R. Perkins
Branch: refs/heads/main
Home: https://github.com/resteasy/resteasy
Commit: cdf0c131894433cb9ee3c48d146aa9c075ae0d64
https://github.com/resteasy/resteasy/commit/cdf0c131894433cb9ee3c48d146aa...
Author: Samuel Cox <sacox(a)paypal.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
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/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
Use getHostName over getHostString when setting RE's URI object.
We want to make a greater effort at capturing the actual hostname used
by the client.
Commit: e095b1db434463b01763d86b1dd1a9d3ebc8d108
https://github.com/resteasy/resteasy/commit/e095b1db434463b01763d86b1dd1a...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java
Log Message:
-----------
Remove a stale comment.
Commit: 08c3f7bdfff2e48e306ef738aa93ad7c128145a1
https://github.com/resteasy/resteasy/commit/08c3f7bdfff2e48e306ef738aa93a...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
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/plugins/server/reactor/netty/UriInfoTest.java
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java
Log Message:
-----------
The request URI should be governed by the client,
which means we need to read it from the Host header. Not sure why
we didn't just copy what the other adapters did. Honestly, some of
that should probably be abstracted _if_ reading from Host header is
truly the right thing to do. I _think_ we shied away from this because
it felt wrong to return 'unknown'. While that still feels wrong, I'm
not sure what is a better option, so copying. I briefly thought about
returning the IP address, which is what was happening before this effort
to _not_ return the IP address!
Commit: 147846e9ca569fbe403adb1600a8c122c69de6ac
https://github.com/resteasy/resteasy/commit/147846e9ca569fbe403adb1600a8c...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
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/plugins/server/reactor/netty/UriInfoTest.java
R server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java
Log Message:
-----------
Use the IP address if the Host header is invalid.
Commit: 26630881f8a1572887a14dbe0edc16cc96e05724
https://github.com/resteasy/resteasy/commit/26630881f8a1572887a14dbe0edc1...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/pom.xml
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/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
Throw IllegalArgumentException if the host address cannot be extracted..
from the request. We discussed 'asserting' here. Maybe that is the
best way. I just haven't seen anyone do that in quite a while.
Commit: 946c998526fad2b121d0f7b64ec089e9eabeaebe
https://github.com/resteasy/resteasy/commit/946c998526fad2b121d0f7b64ec08...
Author: Samuel Cox <crankydillo(a)gmail.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
Remove an empty comment.
Commit: 8a226379884cefc9dcc23cc1258b6ad19eeb4d75
https://github.com/resteasy/resteasy/commit/8a226379884cefc9dcc23cc1258b6...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java
Log Message:
-----------
[RESTEASY-3261] Fix the UriInfoTest.
https://issues.redhat.com/browse/RESTEASY-3261
Signed-off-by: James R. Perkins <jperkins(a)redhat.com>
Commit: 8583e25ac97129d8225f55b4af10b1fc9c1fd1a1
https://github.com/resteasy/resteasy/commit/8583e25ac97129d8225f55b4af10b...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M server-adapters/resteasy-reactor-netty/pom.xml
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/plugins/server/reactor/netty/UriInfoTest.java
R 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:
-----------
Merge pull request #3360 from jamezp/RESTEASY-3261
Use the request's Host header when setting RE's URI object.
Compare: https://github.com/resteasy/resteasy/compare/9bf9c82885f6...8583e25ac971
2 years
[resteasy/resteasy] 09783b: RESTEASY-3259 Proxy Framework fails to Produce Mon...
by James R. Perkins
Branch: refs/heads/main
Home: https://github.com/resteasy/resteasy
Commit: 09783b66f32282c09edb48b8224f4881c83f9464
https://github.com/resteasy/resteasy/commit/09783b66f32282c09edb48b8224f4...
Author: duuude <architect(a)soft4rchitecture.org>
Date: 2022-12-06 (Tue, 06 Dec 2022)
Changed paths:
M resteasy-reactor/src/main/java/org/jboss/resteasy/reactor/MonoRxInvoker.java
M resteasy-reactor/src/main/java/org/jboss/resteasy/reactor/MonoRxInvokerImpl.java
Log Message:
-----------
RESTEASY-3259 Proxy Framework fails to Produce Mono<T> at runtime when proxying a java Interface
The Providers loading mechanism used by RestEasy client actually goes through the Template Parameters types of The Invokers and guessed that for MonoRxInvoker to be called, return type from a remote Interface method shoudl be reactivestreams.Publisher and not Mono<T>. This is because `MonoRxInvoker extends PublisherRxInvoker` which is RxInvoker<Publisher>, instead of `MonoRxInvoker extends RxInvoker<Mono<?>>` like is already the case for resteasy-rxjava2 plugin for example. This causes the Providers loading mechanism to load MonoRxInvoker only if the remote interface signature has Publisher as a return type...
Commit: 0e2d56434f117f4f4cba1b7d81c840d3e7cde963
https://github.com/resteasy/resteasy/commit/0e2d56434f117f4f4cba1b7d81c84...
Author: duuude <architect(a)soft4rchitecture.org>
Date: 2022-12-07 (Wed, 07 Dec 2022)
Changed paths:
M resteasy-reactor/src/main/java/org/jboss/resteasy/reactor/MonoRxInvokerImpl.java
Log Message:
-----------
RESTEASY-3259 fixes review comment
Commit: 721094f8ec1d6ace600737070015a299ec799223
https://github.com/resteasy/resteasy/commit/721094f8ec1d6ace600737070015a...
Author: duuude <architect(a)soft4rchitecture.org>
Date: 2022-12-07 (Wed, 07 Dec 2022)
Changed paths:
A resteasy-reactor/src/test/java/org/jboss/resteasy/reactor/proxyframework/CustomResource.java
A resteasy-reactor/src/test/java/org/jboss/resteasy/reactor/proxyframework/MonoWithProxyFrameworkApiTest.java
Log Message:
-----------
RESTEASY-3259 adds Proxy Framework test with Mono<T> in remote service interface
Commit: ad0078ebed320c46228fb6fdd636705e214e2941
https://github.com/resteasy/resteasy/commit/ad0078ebed320c46228fb6fdd6367...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M resteasy-reactor/src/main/java/org/jboss/resteasy/reactor/MonoRxInvoker.java
M resteasy-reactor/src/main/java/org/jboss/resteasy/reactor/MonoRxInvokerImpl.java
A resteasy-reactor/src/test/java/org/jboss/resteasy/reactor/proxyframework/CustomResource.java
A resteasy-reactor/src/test/java/org/jboss/resteasy/reactor/proxyframework/MonoWithProxyFrameworkApiTest.java
Log Message:
-----------
Merge pull request #3348 from soft4rchitecture/RESTEASY-3259/fixes-error-when-mono-in-remote-interface
RESTEASY-3259 Proxy Framework fails to Produce Mono<T> at runtime whe…
Compare: https://github.com/resteasy/resteasy/compare/c03523fa14c5...ad0078ebed32
2 years
[resteasy/resteasy] a58179: [RESTEASY-2874] Web Target Proxy not overriding co...
by James R. Perkins
Branch: refs/heads/5.0
Home: https://github.com/resteasy/resteasy
Commit: a581790e18fdd1287b8d56dd3be6afc011b9594f
https://github.com/resteasy/resteasy/commit/a581790e18fdd1287b8d56dd3be6a...
Author: petrberan <Peta.Beran(a)seznam.cz>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/internal/ClientRequestHeaders.java
M testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/core/servlet/ServletConfigTest.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/param/UserDefinedHeaderParamTest.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/param/resource/UserDefinedHeaderParamResource.java
Log Message:
-----------
[RESTEASY-2874] Web Target Proxy not overriding content type header
Signed-off-by: James R. Perkins <jperkins(a)redhat.com>
Commit: 6a7c191ce6c25b02e2690d6308f17170554cbdf4
https://github.com/resteasy/resteasy/commit/6a7c191ce6c25b02e2690d6308f17...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/internal/ClientRequestHeaders.java
M testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/param/UserDefinedHeaderParamTest.java
M testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/param/resource/UserDefinedHeaderParamResource.java
Log Message:
-----------
[RESTEASY-2874] Follow up on setting the content type for proxies. Ensure the test uses the correct method.
Signed-off-by: James R. Perkins <jperkins(a)redhat.com>
Commit: 6149edbe0ce2faaa4e4898f4919767109434d726
https://github.com/resteasy/resteasy/commit/6149edbe0ce2faaa4e4898f491976...
Author: James R. Perkins <jperkins(a)redhat.com>
Date: 2022-12-12 (Mon, 12 Dec 2022)
Changed paths:
M resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/internal/ClientRequestHeaders.java
M testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/core/servlet/ServletConfigTest.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/param/UserDefinedHeaderParamTest.java
A testsuite/integration-tests/src/test/java/org/jboss/resteasy/test/resource/param/resource/UserDefinedHeaderParamResource.java
Log Message:
-----------
Merge pull request #3358 from jamezp/RESTEASY-2874-5.x
[RESTEASY-2874] Web Target Proxy not overriding content type headerh
Compare: https://github.com/resteasy/resteasy/compare/a758a6d47e00...6149edbe0ce2
2 years