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

Skip access check on absent will queue (backport #11023) #11024

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Apr 17, 2024

Resolves #11021

Prior to this commit, an MQTT client that connects to RabbitMQ needed configure access to its will queue even if the will queue has never existed. This breaks client apps connecting with either v3 or v4 or with v5 without making use of the Will-Delay-Interval.

Specifically, in 3.13.0 and 3.13.1 an MQTT client that connects to RabbitMQ needs unnecessarily configure access to queue mqtt-will-<MQTT client ID>.

This commit only check for configure access, if the queue actually gets deleted, i.e. if it existed.


This is an automatic backport of pull request #11023 done by Mergify.

Resolves https://github.com/rabbitmq/rabbitmq-server/discussions/11021

Prior to this commit, an MQTT client that connects to RabbitMQ needed
configure access to its will queue even if the will queue has never
existed. This breaks client apps connecting with either v3 or v4 or with
v5 without making use of the Will-Delay-Interval.

Specifically, in 3.13.0 and 3.13.1 an MQTT client that connects to
RabbitMQ needs unnecessarily configure access to queue
`mqtt-will-<MQTT client ID>`.

This commit only check for configure access, if the queue actually gets
deleted, i.e. if it existed.

(cherry picked from commit bb106ff)
@mergify mergify bot assigned ansd Apr 17, 2024
@ansd ansd merged commit 9115aff into v3.13.x Apr 17, 2024
21 checks passed
@ansd ansd deleted the mergify/bp/v3.13.x/pr-11023 branch April 17, 2024 11:38
@michaelklishin michaelklishin added this to the 3.13.2 milestone Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants