logstash-output-jms 3.0.3 → 3.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc40b0019682516e29929ca3cff34388aca968efb37d45bdc293e86d8bc0e2c3
-  data.tar.gz: 93cf282e0e42ed4abe5ad3975f24e602294992f0cd7c2934416f7dda40104ab7
+  metadata.gz: e8c6884ef9fed2c5e1f52d37d32f01bcb1521381cfb1b8355a80483d6004e39a
+  data.tar.gz: 83dcf48a5d1851be6c39b55992f98c72140dab4fd55046b037a328d5a952b25b
 SHA512:
-  metadata.gz: a7ba4161f4f739edebd5e205e47ecb4cd9f9adaf46de0989777a2ee82f8a6855f394dcd5a9dd559d78118cf789eb123c4cc3fe7f4c7c8b8adaa54a577e6d6e1a
-  data.tar.gz: 9a7b0913a86792e0ac8f65e516272d90bf152853db3d901c370d29bfac10702508baa2b6ba5228089a61751ff6a6a8eb901c3b2b581501c6d9d1fd22ad8a0824
+  metadata.gz: b0b8b4672024ffa12932af9562dfc7e365b818624887893317642a0bf40eceb8cb881787168e00e715adcd6908b035f4278d571b9028773bdde769b0dfc86fcf
+  data.tar.gz: 296bf31b7e15bf1ca755060b94f62f59ede467132ed363d22fcb736d5a2aeee8d03f82735b27b8dab810307b027fc9fa406e7688d8ff2a0962644beaa5b5fd79

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 3.0.4
+  - Fixes an issue where `delivery_mode` directive was silently ignored.
+
 ## 3.0.3
   - Docs: Set the default_codec doc attribute.
 

data/lib/logstash/outputs/jms.rb

@@ -23,7 +23,7 @@ class LogStash::Outputs::Jms < LogStash::Outputs::Base
 
 # Name of delivery mode to use
 # Options are "persistent" and "non_persistent" if not defined nothing will be passed.
-config :delivery_mode, :validate => :string, :default => nil
+config :delivery_mode, :validate => %w(persistent non_persistent)
 
 # If pub-sub (topic) style should be used or not.
 # Mandatory
@@ -95,16 +95,27 @@ config :jndi_context, :validate => :hash
     end
 
     @logger.debug("JMS Config being used", :context => @jms_config)
-    @connection = JMS::Connection.new(@jms_config)
+    begin
+      # The jruby-jms adapter dynamically loads the Java classes that it extends, and may fail
+      @connection = JMS::Connection.new(@jms_config)
+    rescue NameError => ne
+      if @require_jars && !@require_jars.empty?
+        logger.warn('The `require_jars` directive was provided, but may not correctly map to a JNS provider', :require_jars => @require_jars)
+      end
+      logger.error('Failed to load JMS Connection, likely because a JMS Provider is not on the Logstash classpath '+
+                   'or correctly specified by the plugin\'s `require_jars` directive', :exception => ne.message, :backtrace => ne.backtrace)
+      fail(LogStash::PluginLoadingError, 'JMS Input failed to load, likely because a JMS provider was not available')
+    end
+
     @session = @connection.create_session()
 
     # Cache the producer since we should keep reusing this one.
     destination_key = @pub_sub ? :topic_name : :queue_name
     @producer = @session.create_producer(@session.create_destination(destination_key => @destination))
 
-    if !@delivery_mode.nil?
-      @producer.delivery_mode_sym = @deliver_mode
-    end
+    # If a delivery mode has been specified, inform the producer
+    @producer.delivery_mode_sym = @delivery_mode.to_sym unless @delivery_mode.nil?
+
   end # def register
 
   def receive(event)

data/logstash-output-jms.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
 	s.name            = 'logstash-output-jms'
-	s.version         = '3.0.3'
+	s.version         = '3.0.4'
 	s.licenses        = ['Apache License (2.0)']
 	s.summary         = "Push events to a JMS topic or queue."
 	s.description     = "This gem is a Logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/logstash-plugin install gemname. This gem is not a stand-alone program"

data/spec/outputs/jms_spec.rb

@@ -14,6 +14,7 @@ describe "outputs/jms" do
 
   context 'when initializing' do
     it "should register" do
+      pending('JMS Provider')
       output = LogStash::Plugin.lookup("output", "jms").new(jms_config)
       expect {output.register}.to_not raise_error
     end
@@ -26,6 +27,7 @@ describe "outputs/jms" do
 
   context 'when outputting messages' do
     it 'should send logstash event to jms queue' do
+      pending('JMS Provider')
       jms = LogStash::Outputs::Jms.new(jms_config)
       jms.register
       jms.receive(event)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-jms
 version: !ruby/object:Gem::Version
-  version: 3.0.3
+  version: 3.0.4
 platform: ruby
 authors:
 - Elasticsearch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-04-06 00:00:00.000000000 Z
+date: 2018-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -125,7 +125,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project:
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key:
 specification_version: 4
 summary: Push events to a JMS topic or queue.