Skip to content

Conversation

@mbengtsson
Copy link
Contributor

@mbengtsson mbengtsson commented Aug 15, 2025

Centralize and unify address translation for producers/consumers to preserve \ in names and avoid mismatches. Keeps temp/advisory/FQQN/ VirtualTopic behavior unchanged and adds tests for all cases.

Changes include:

  • Replace ad-hoc wildcard and escape handling in AMQSession, AMQConsumer, and OpenWireConnection with central OpenWireUtil methods.
  • Preserve temp/advisory destinations, FQQN structure, and VirtualTopic consumer naming conventions.
  • Strip matching queue:// or topic:// scheme prefixes only when type matches.
  • Add unit and integration tests covering escape rules, wildcard conversion, FQQN handling, and advisory/temp queue cases.
  • Remove obsolete AMQSession.convertWildcard().

I think this change is 100% backward compatible and requires no client changes.

Marcus Bengtsson added 2 commits August 15, 2025 15:56
Centralize and unify address translation for producers/consumers to
preserve '\' in names and avoid mismatches. Keeps temp/advisory/FQQN/
VirtualTopic behavior unchanged and adds tests for all cases.
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.

1 participant