fluent-plugin-input-gelf 0.2.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: eee1b338888b0ca1ec6f85012c47afbf14266fd4
-  data.tar.gz: 7d0557b52f33d68de6e1469f9e184a96e43de44d
+SHA256:
+  metadata.gz: 8f097e210aed3af36c1e2d4e69863ae9e7def76a8dc146b1d62142e7a468021d
+  data.tar.gz: 12fc71512e8cfd1a77bf061caf6857e2b8da1489a0050054dac7071a507ffb10
 SHA512:
-  metadata.gz: bb91d66ad6039147e5f795a9037e61f41541e7f1b0309ff0d0ebcc8ef15f0dbbdc6e2437f24254bbad67cec5c61da7fcf8f7c67cbf3535ff8d7fc5ee19b0f967
-  data.tar.gz: b69d7aa70c1b58b28604c8c3f566197e4ded4928975c6a07d2be38c90a837a908a4c0f9e3b9cbfd55b74b326b1c20df011957a4faeb16f9495ad05e6be003f4f
+  metadata.gz: 8811bb2fb2d6d81120d7007d7ca939fd065f22ae0d6ba41f6bd6279c74e8e3cc6926f7b82c441f73f33a784de3c965d59fc5f940721b344b0a36a5b80ef668fd
+  data.tar.gz: 04f05ae7694cadf96b89d6ed249e03d10228be6282cfff4652a0347414bd634b49222f62d472594594e1c77b4bc43be10b2894343ef406fe5a8831482be1ef81

data/.travis.yml

@@ -1,16 +1,33 @@
 language: ruby
 
 rvm:
-  - 2.1
-  - 2.2.2
-  - 2.3.0
-  - 2.4.0
+- 2.1
+- 2.2
+- 2.3
+- 2.4
+- 2.5
 
 gemfile:
-  - Gemfile
+- Gemfile
 
 branches:
   only:
-    - master
+  - master
+  # Tagged commits
+  - /^v\d+\.\d+(\.\d+)?(-\S*)?$/
 
 script: bundle exec rake test
+
+jobs:
+  include:
+  - stage: gem release
+    rvm: 2.3
+    script: echo "Deploying to rubygems.org ..."
+    deploy:
+      provider: rubygems
+      api_key:
+        secure: R06ICCYHjZ+bk+CocD+eZROiTsfDnmUlugdSIqND0f9Z+Vi8ALOZmIEhlpBvloRaOLU0pz3d9juj74vHMJ1boJN6Qgn+CtAGSDzesxm6vcd/AP87mEzAaJvvHRme7bNLy7gt1RvrmVzDjyOe2fF8kcVAz19bEArJ1rqBXeLiLY+fOPenCsWRNrOt2+yOH5qRgdFs+n2NQ6oQrDEq18uZJVQ74WnSyKGylIhPfXj+TcgMrDGJtxqWYmL+DeuzOVk73REMHiw+9ljvhjVIdMxiU/oXge9y7Ua7q6OlkJgmChaWrs06bHiwhJHz6yh+OXebVFbxHTsPdsw4w+Uv9/MB961zgR58yLVi6AJr5xosM8l92S/OJeG9paO43xuQGeX+mch7m/827VbeM8aLtRGXjNILC3BW+l9O++Yg0eF6YC4c6NobNOX4ZpfGNopDsoyYI19Yj9SoKvoqw3WtAZsX2T5CrAdG1PHLKNDwyCbuiYjlNKAAHy5ZCB7X63JrDAKgwUQSnYEPEGNVzisnNfyqVxQnvTDZLwgzWcYxTBCjXoO09+qsbB9lpgecT45UI80VgW70nokEzilmyXARG+eBGKhz2iJli+mrdZbpfzws/PD/Hr5dubgn8mNX5/e6DRMoTr9xQOGrvdXhET+ygiTEWg1VMzfcZ2WpeJTveV9h2BQ=
+      gem: fluent-plugin-input-gelf
+      on:
+        tags: true
+        repo: MerlinDMC/fluent-plugin-input-gelf

data/fluent-plugin-input-gelf.gemspec

@@ -7,7 +7,7 @@ Gem::Specification.new do |gem|
   gem.license     = "MIT"
   gem.homepage    = "https://github.com/MerlinDMC/fluent-plugin-input-gelf"
   gem.summary     = gem.description
-  gem.version     = "0.2.0"
+  gem.version     = "0.3.1"
   gem.authors     = ["Daniel Malon"]
   gem.email       = "daniel.malon@me.com"
   gem.has_rdoc    = false
@@ -16,7 +16,7 @@ Gem::Specification.new do |gem|
   gem.executables = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   gem.require_paths = ['lib']
 
-  gem.add_runtime_dependency "fluentd"
+  gem.add_runtime_dependency "fluentd", [">= 0.14.10", "< 2"]
   gem.add_runtime_dependency "gelfd2", ">= 0.3.0"
 
   gem.add_development_dependency "bundler"

data/lib/fluent/plugin/in_gelf.rb

@@ -4,68 +4,52 @@ require 'time'
 require 'cool.io'
 require 'yajl'
 
-require 'fluent/input'
+require 'gelfd2'
 
-module Fluent
-  class GelfInput < Fluent::Input
+require 'fluent/plugin/input'
+
+module Fluent::Plugin
+  class GelfInput < Fluent::Plugin::Input
     Fluent::Plugin.register_input('gelf', self)
 
+    helpers :server, :parser, :compat_parameters
+
+    DEFAULT_PARSER = 'json'.freeze
+
     def initialize
       super
-      require 'fluent/plugin/socket_util'
-      require 'gelfd2'
     end
 
     desc "The value is the tag assigned to the generated events."
     config_param :tag, :string
-    desc 'The format of the payload.'
-    config_param :format, :string, default: 'json'
     desc 'The port to listen to.'
     config_param :port, :integer, default: 12201
     desc 'The bind address to listen to.'
     config_param :bind, :string, default: '0.0.0.0'
     desc 'The transport protocol used to receive logs.(udp, tcp)'
-    config_param :protocol_type, default: :udp do |val|
-      case val.downcase
-      when 'tcp'
-        :tcp
-      when 'udp'
-        :udp
-      else
-        raise ConfigError, "gelf input protocol type should be 'tcp' or 'udp'"
-      end
-    end
-    config_param :blocking_timeout, :time, default: 0.5
+    config_param :protocol_type, :enum, list: [:udp, :tcp], default: :udp
     desc 'Strip leading underscore'
     config_param :strip_leading_underscore, :bool, default: true
 
+    config_section :parse do
+      config_set_default :@type, DEFAULT_PARSER
+    end
+
     def configure(conf)
+      compat_parameters_convert(conf, :parser)
       super
 
-      @parser = Plugin.new_parser(@format)
-      @parser.configure(conf)
+      @parser = parser_create
     end
 
     def start
-      @loop = Coolio::Loop.new
-      @handler = listen(method(:receive_data))
-      @loop.attach(@handler)
+      super
 
-      @thread = Thread.new(&method(:run))
+      listen
     end
 
     def shutdown
-      @loop.watchers.each { |w| w.detach }
-      @loop.stop
-      @handler.close
-      @thread.join
-    end
-
-    def run
-      @loop.run(@blocking_timeout)
-    rescue
-      log.error 'unexpected error', error: $!.to_s
-      log.error_backtrace
+      super
     end
 
     def receive_data(data, addr)
@@ -86,7 +70,7 @@ module Fluent
         end
 
         # Use the recorded event time if available
-        time = record.delete('timestamp').to_i if record.key?('timestamp')
+        time = record.delete('timestamp').to_f if record.key?('timestamp')
 
         # Postprocess recorded event
         strip_leading_underscore_(record) if @strip_leading_underscore
@@ -98,14 +82,16 @@ module Fluent
       log.error_backtrace
     end
 
-    def listen(callback)
+    def listen
       log.info "listening gelf socket on #{@bind}:#{@port} with #{@protocol_type}"
       if @protocol_type == :tcp
-        Coolio::TCPServer.new(@bind, @port, SocketUtil::TcpHandler, log, "\n", callback)
+        server_create(:in_tcp_server, @port, bind: @bind) do |data, conn|
+          receive_data(data, conn)
+        end
       else
-        @usock = SocketUtil.create_udp_socket(@bind)
-        @usock.bind(@bind, @port)
-        SocketUtil::UdpHandler.new(@usock, log, 8192, callback)
+        server_create(:in_udp_server, @port, proto: :udp, bind: @bind, max_bytes: 8192) do |data, sock|
+          receive_data(data, sock)
+        end
       end
     end
 

data/test/plugin/test_in_gelf.rb

@@ -20,7 +20,7 @@ class GelfInputTest < Test::Unit::TestCase
   !
 
   def create_driver(conf)
-    Fluent::Test::InputTestDriver.new(Fluent::GelfInput).configure(conf)
+    Fluent::Test::Driver::Input.new(Fluent::Plugin::GelfInput).configure(conf)
   end
 
   def test_configure
@@ -31,7 +31,7 @@ class GelfInputTest < Test::Unit::TestCase
       d = create_driver(v)
       assert_equal PORT, d.instance.port
       assert_equal k, d.instance.bind
-      assert_equal 'json', d.instance.format
+      assert_equal 'json', d.instance.parser_configs.first["@type"]
     }
   end
 
@@ -48,22 +48,19 @@ class GelfInputTest < Test::Unit::TestCase
         {:short_message => 'short message', :full_message => 'full message', :timestamp => 12345678.12345}
       ]
 
-      d.run do
+      d.run(expect_emits: 2)  do
         n = GELF::Notifier.new(k, PORT)
 
         tests.each { |test|
           n.notify!(test)
         }
-
-        sleep 1
       end
 
-      emits = d.emits
+      emits = d.events
       assert_equal tests.length, emits.length, 'missing emitted events'
       emits.each_index { |i|
-        puts emits[i].to_s
         assert_equal 'gelf', emits[i][0]
-        assert_equal tests[i][:timestamp].to_i, emits[i][1] unless tests[i][:timestamp].nil?
+        assert_equal tests[i][:timestamp].to_f, emits[i][1] unless tests[i][:timestamp].nil?
         assert_equal tests[i][:short_message], emits[i][2]['short_message']
         assert_equal tests[i][:full_message], emits[i][2]['full_message']
       }
@@ -95,22 +92,19 @@ class GelfInputTest < Test::Unit::TestCase
         }
       ]
 
-      d.run do
+      d.run(expect_emits: 1) do
         n = GELF::Notifier.new(k, PORT)
 
         tests.each { |test|
           n.notify!(test[:given])
         }
-
-        sleep 1
       end
 
-      emits = d.emits
+      emits = d.events
       assert_equal tests.length, emits.length, 'missing emitted events'
       emits.each_index { |i|
-        puts emits[i].to_s
         assert_equal 'gelf', emits[i][0]
-        assert_equal tests[i][:timestamp].to_i, emits[i][1] unless tests[i][:timestamp].nil?
+        assert_equal tests[i][:timestamp].to_f, emits[i][1] unless tests[i][:timestamp].nil?
         assert_block "expectation not met: #{tests[i][:expected]}" do
           emits[i][2].merge(tests[i][:expected]) == emits[i][2]
         end

data/test/test_helper.rb

@@ -13,16 +13,7 @@ $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 
 require 'fluent/test'
-unless ENV.has_key?('VERBOSE')
-  nulllogger = Object.new
-  nulllogger.instance_eval {|obj|
-    def method_missing(method, *args)
-      # pass
-    end
-  }
-  $log = nulllogger
-end
-
+require 'fluent/test/driver/input'
 require 'fluent/plugin/in_gelf'
 
 def unused_port

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-input-gelf
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Daniel Malon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-11 00:00:00.000000000 Z
+date: 2019-04-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.14.10
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.14.10
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: gelfd2
   requirement: !ruby/object:Gem::Requirement
@@ -129,11 +135,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: A GELF input plugin for fluentd
-test_files:
-- test/plugin/test_in_gelf.rb
-- test/test_helper.rb
+test_files: []