Skip to content

Reject unconvertible messages in AMQP 1.0 inbound endpoints - #11500

Merged
artembilan merged 1 commit into
spring-projects:mainfrom
janwaleed09:amqp10-conversion-failure
Sep 29, 2026
Merged

artembilan merged 1 commit into
spring-projects:mainfrom
janwaleed09:amqp10-conversion-failure

Conversation

@janwaleed09

@janwaleed09 janwaleed09 commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

IntegrationRabbitAmqpMessageListener converts the message before its try block, so a MessageConversionException reaches the container's ConditionalRejectingErrorHandler unwrapped and is not treated as fatal.

https://docs.spring.io/spring-integration/reference/amqp/conversion-inbound.html

Signed-off-by: Jan Mohr <jan.mohr.1@web.de>
@janwaleed09
janwaleed09 marked this pull request as ready for review September 29, 2026 13:59
@artembilan artembilan added this to the 7.2.0-RC1 milestone Sep 29, 2026
@artembilan
artembilan merged commit adafb92 into spring-projects:main Sep 29, 2026
3 checks passed
spring-builds pushed a commit that referenced this pull request Sep 29, 2026
Fixes: #11500

The `IntegrationRabbitAmqpMessageListener` converts the message before its try block,
so a `MessageConversionException` reaches the container's `ConditionalRejectingErrorHandler` unwrapped and is not treated as `fatal`.

Signed-off-by: Jan Mohr <jan.mohr.1@web.de>

(cherry picked from commit adafb92)
artembilan pushed a commit that referenced this pull request Sep 29, 2026
Fixes: #11500

The `IntegrationRabbitAmqpMessageListener` converts the message before its try block,
so a `MessageConversionException` reaches the container's `ConditionalRejectingErrorHandler` unwrapped and is not treated as `fatal`.

Signed-off-by: Jan Mohr <jan.mohr.1@web.de>

(cherry picked from commit adafb92)

# Conflicts:
#	spring-integration-amqp/src/test/java/org/springframework/integration/amqp/inbound/AmqpClientMessageProducerTests.java
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.

3 participants