search

jentrata / jentrata-msh

Jentrata - Message Handler Service
jentrara.org
Other
19 stars 57 forks source link

added custom JMS connection factory for inbound ebms messages #38

Open EdwinVDS opened 7 years ago

EdwinVDS commented 7 years ago

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

base2bot commented 7 years ago

Can one of the admins verify this patch?

aaronwalker commented 7 years ago

@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 commented 7 years ago

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.