Skip to content

Conversation

@artembilan
Copy link
Member

The AbstractMessageProducingHandler has async mode, which is handled by the reply payload type.
However, the AbstractTransformer produces the whole Message with that async reply payload.

  • Fix AbstractMessageProducingHandler to check for a Message type on a reply object. Extract its payload for the async logic.
    And recreate the final message for reply after async container fulfilment
  • Disable WebFluxDslTests.testHttpReactivePostWithError() as failing against the latest SF

Auto-cherry-pick to 6.5.x & 6.4.x

The `AbstractMessageProducingHandler` has `async` mode,
which is handled by the reply payload type.
However, the `AbstractTransformer` produces the whole `Message` with that async reply payload.

* Fix `AbstractMessageProducingHandler` to check for a `Message` type on a reply object.
Extract its payload for the async logic.
And recreate the final message for reply after async container fulfilment
* Disable `WebFluxDslTests.testHttpReactivePostWithError()` as failing against the latest SF

**Auto-cherry-pick to `6.5.x` & `6.4.x`**
Copy link
Contributor

@cppwfs cppwfs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Nice Catch!

@cppwfs cppwfs merged commit 4f35761 into spring-projects:main Dec 15, 2025
2 of 6 checks passed
spring-builds pushed a commit that referenced this pull request Dec 15, 2025
The `AbstractMessageProducingHandler` has `async` mode,
which is handled by the reply payload type.
However, the `AbstractTransformer` produces the whole `Message` with that async reply payload.

* Fix `AbstractMessageProducingHandler` to check for a `Message` type on a reply object.
Extract its payload for the async logic.
And recreate the final message for reply after async container fulfilment
* Disable `WebFluxDslTests.testHttpReactivePostWithError()` as failing against the latest SF

(cherry picked from commit 4f35761)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants