Skip to content

AMQ-9475 - ConsumerControl commands should not auto create wildcard dests#1200

Merged
cshannon merged 1 commit intoapache:mainfrom
cshannon:AMQ-9475
Apr 11, 2024
Merged

AMQ-9475 - ConsumerControl commands should not auto create wildcard dests#1200
cshannon merged 1 commit intoapache:mainfrom
cshannon:AMQ-9475

Conversation

@cshannon
Copy link
Contributor

This fixes an issue where wildcard destinations could be inadvertently created by ConsumerControl commands which could lead to problems. The processing logic for this command now will only look up existing destinations if they exist.

dests

This fixes an issue where wildcard destinations could be inadvertently
created by ConsumerControl commands which could lead to problems. The
processing logic for this command now will only look up existing
destinations if they exist.
Copy link
Contributor

@mattrpav mattrpav left a comment

Choose a reason for hiding this comment

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

LGTM (pending CI tests passing)

@cshannon
Copy link
Contributor Author

I'll let the tests finish and then I'll merge and backport.

@thezbyg
Copy link
Contributor

thezbyg commented Apr 10, 2024

@cshannon @mattrpav I rebuilt ActiveMQ with these changes and can confirm that wildcard topics are no longer created and everything works as expected.

Thank you.

@cshannon
Copy link
Contributor Author

@cshannon @mattrpav I rebuilt ActiveMQ with these changes and can confirm that wildcard topics are no longer created and everything works as expected.

Thank you.

Thanks for the quick test, glad that worked for you. This will be included in the next bug fix releases for 6.1.x and 5.18.x. 5.18.4 is already under vote so this will be in 5.18.5 but obviously you can just build your own version as you did.

@cshannon cshannon merged commit 78d9555 into apache:main Apr 11, 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.

3 participants