Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid an exception when an AMQP 0-9-1-originating message with expiration set is converted for an MQTT consumer (backport #12710) #12711

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Nov 13, 2024

when an AMQP 0-9-1 publisher publishes a message
that has expiration set.

Stack trace was contributed in #12707 by @rdsilio.

Originally submitted against v4.0.x as #12708.

To reproduce the exception:

  1. Start a node with rabbitmq_mqtt enabled
  2. Tail its logs
  3. Install github.com/rabbitmq/omq
  4. omq amqp-mqtt --message-ttl 10s -t /topic/my_topic -T my_topic -C 1
    This is an automatic backport of pull request 4.1: Avoid an exception when an AMQP 0-9-1-originating message with expiration set is converted for an MQTT consumer #12710 done by Mergify.

…xpiration set is converted for an MQTT consumer (#12710)

* MQTT: avoid an exception

when an AMQP 0-9-1 publisher publishes a message
that has expiration set.

Stack trace was contributed in #12707 by @rdsilio.

* mc_mqtt_SUITE test for #12707 #12710

* MQTT protocol_interop_SUITE: new test for #12710 #12707

* Simplify tests

---------

Co-authored-by: David Ansari <david.ansari@gmx.de>
(cherry picked from commit c78bc8a)
@ansd ansd changed the title 4.1: Avoid an exception when an AMQP 0-9-1-originating message with expiration set is converted for an MQTT consumer (backport #12710) Avoid an exception when an AMQP 0-9-1-originating message with expiration set is converted for an MQTT consumer (backport #12710) Nov 13, 2024
@ansd ansd merged commit db25654 into v4.0.x Nov 13, 2024
195 checks passed
@ansd ansd deleted the mergify/bp/v4.0.x/pr-12710 branch November 13, 2024 09:02
@ansd ansd added this to the 4.0.4 milestone Nov 13, 2024
@ansd ansd added the bug label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants