Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

fix connector to work with all addresses #44

Merged
merged 1 commit into from
Aug 12, 2020
Merged

fix connector to work with all addresses #44

merged 1 commit into from
Aug 12, 2020

Conversation

krancour
Copy link
Contributor

@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
Copy link

DanSalt commented Aug 10, 2020

@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 #29

@vromero
Copy link
Owner

vromero commented Aug 12, 2020

Thanks @DanSalt @krancour

@vromero vromero merged commit 60d67b5 into vromero:master Aug 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants