fluent-plugin-amqp 0.9.0 → 0.9.1

data/lib/fluent/plugin/out_amqp.rb

@@ -11,6 +11,11 @@ module Fluent
 
     attr_accessor :connection
 
+    #Attribute readers to support testing
+    attr_reader :exch
+    attr_reader :channel
+
+
     config_param :host, :string, :default => nil
     config_param :hosts, :array, :default => nil
     config_param :user, :string, :default => "guest"
@@ -83,9 +88,15 @@ module Fluent
 
     def write(chunk)
       chunk.msgpack_each do |(tag, time, data)|
-        data = JSON.dump( data ) unless data.is_a?( String )
-        log.debug "Sending message #{data}, :key => #{routing_key( tag)} :headers => #{headers(tag,time)}"
-        @exch.publish(data, :key => routing_key( tag ), :persistent => @persistent, :headers => headers( tag, time ))
+        begin
+           data = JSON.dump( data ) unless data.is_a?( String )
+           log.debug "Sending message #{data}, :key => #{routing_key( tag)} :headers => #{headers(tag,time)}"
+           @exch.publish(data, :key => routing_key( tag ), :persistent => @persistent, :headers => headers( tag, time ))
+        rescue JSON::GeneratorError => e
+           log.error "Failure converting data object to json string: #{e.message}"
+           # Debug only - otherwise we may pollute the fluent logs with unparseable events and loop
+           log.debug "JSON.dump failure converting [#{data}]"
+        end
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-amqp
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.1
   prerelease: 
 platform: ruby
 authors:
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-30 00:00:00.000000000 Z
+date: 2016-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd
@@ -183,7 +183,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 803931007263288493
+      hash: 337161649261298881
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: