fix connector to work with all addresses

vromero / activemq-artemis-helm

Helm chart for a cluster of ActiveMQ Artemis (Work in progress)

42 stars 68 forks source link

fix connector to work with all addresses #44

Closed krancour closed 4 years ago

krancour commented 4 years ago

@vromero nice chart!

I spent a chunk of time this weekend trying to figure out why messages weren't being properly distributed and redistributed. I eventually discovered the line that this PR proposes to remove constrains the connector to only working for addresses beginning jms-- which I assume was not the intended behavior.

DanSalt commented 4 years ago

@vromero I can confirm that this PR does fix the problem. The "jms" comes directly from the Artemis documentation, but I definitely don't see any rationale in restricting it to that by default.

This is the same as https://github.com/vromero/activemq-artemis-helm/pull/29

vromero commented 4 years ago

Thanks @DanSalt @krancour