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 #11023

Merged
merged 1 commit into from
Apr 17, 2024
Merged

Conversation

ansd
Copy link
Member

@ansd ansd 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.

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.
@ansd ansd merged commit bb106ff into main Apr 17, 2024
21 checks passed
@ansd ansd deleted the mqtt-will-queue-configure-check branch April 17, 2024 10:50
@ansd ansd mentioned this pull request Apr 22, 2024
ansd added a commit that referenced this pull request Apr 22, 2024
Fix the failing GitHub action Test Authentication/Authorization backends
via mutiple messaging protocols / selenium originally caused by #11023
ansd added a commit that referenced this pull request Apr 22, 2024
Fix the failing GitHub action Test Authentication/Authorization backends
via mutiple messaging protocols / selenium originally caused by #11023
mergify bot pushed a commit that referenced this pull request Apr 22, 2024
Fix the failing GitHub action Test Authentication/Authorization backends
via mutiple messaging protocols / selenium originally caused by #11023

(cherry picked from commit 6e056e5)
ansd added a commit that referenced this pull request Apr 22, 2024
Fix the failing GitHub action Test Authentication/Authorization backends
via mutiple messaging protocols / selenium originally caused by #11023

(cherry picked from commit 6e056e5)
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