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

added custom JMS connection factory for inbound ebms messages #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EdwinVDS
Copy link
Contributor

Pull request with isolated change for 2.x branch. Enables deployment of other JMS clients than ActiveMQ

@base2bot
Copy link
Collaborator

Can one of the admins verify this patch?

@aaronwalker
Copy link
Member

@EdwinVDS basically in JMS plugin is kind of deprecated in favour of the new camel integration that I added as part of 3.x. So basically you can integrate jentrata with pretty anything via camel. If you want I can merge this for now but I might choose to remove the JMS support in order to make it less confusing on how you integrate with jentrata.

Would be keen to get your feedback on this integration (https://github.com/jentrata/jentrata-msh/tree/master/Plugins/jentrata.integration) there is an example integration here (https://github.com/jentrata/jentrata-msh/blob/master/Dist/src/main/deploy/example-ebms.xml.example)

@EdwinVDS
Copy link
Contributor Author

Hi @aaronwalker . Since we're using the 2.x branch in deployment we needed this change. I did see the Camel update but we were too far down the test-cycle for a migration to the 3.x branch. If you could merge this into 2.x that would be great.

Also, will have a look at the link and get back to you.

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.

None yet

3 participants