Change stomp client library

logstash-plugins / logstash-input-stomp

Apache License 2.0

3 stars 16 forks source link

Open wiibaa opened 8 years ago

wiibaa commented 8 years ago

I'm wondering if it would be a good idea to change client library to NEW => https://github.com/stompgem/stomp OLD => https://github.com/iande/onstomp

Pros:

onstomp has very limited maintenance power https://github.com/iande/onstomp/issues/20
stompgem/stomp support STOMP 1.2
stompgem/stomp has good example on configuring ssl and certs

Cons: I do not see strong Cons unless the heavy work on re-testing this plugin against your Stomp infra.

Any comments are warmly welcomed !

LionelCons commented 8 years ago

+1 for stompgem/stomp which is used in other projects like MCollective

nicolargo commented 6 years ago

+1 for stompgem. STOMP 1.0 do not support any heart-beat frame. It is a mess to connect to an Artemis/STOMP topic...