fluent-plugin-gelf-best 1.3.2 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7327c66aa6fddfe60ed1448e76a59ecd906b458957758b748ee3162aefa85d0f
-  data.tar.gz: 2f5b1820ab88ce8665288b2e218e8be53aa274d5047e32cde33faaa5dad967c6
+  metadata.gz: dc3a134e34dfdc0c22a93f90ca53b655bbb9133f787726adb478ffdb6b74bba3
+  data.tar.gz: f65ca691ac9b2e7046878b1b8b6315a21c6a3d575408ece21aff885c0f3c3add
 SHA512:
-  metadata.gz: 2735ce20d4dc4405d137405a30ec2c52dc83343fae9fdb7312f35e1b671546bd95ce0ab1c93648deaa1e0175fcedb79df739596aaedb27e767135695169a1ec5
-  data.tar.gz: 4add4d31fc40b78c7ff70450b556749e755f0fa6072e62a24160c4afa8ae63f09f4a46c2e2a7d26e876bb9379b65264787da37a603aa807e5dd0489c092416af
+  metadata.gz: f2adf1721b08d85f7a191d7121eb0a0190aee674f70159c828a475a4f717a90594307e116b1a88212d0095c9b546d48134c596e7fb186f6dba9cc0476905e616
+  data.tar.gz: 7bca600938707cda06ce79d798d11dd3dd3756ee7a0a3016b2770aa9a63d1133f0a7bcb745d8e14699d6503099cced2d6ff2a63b008731e8aa8a4fc3b6a652a4

data/fluent-plugin-gelf-best.gemspec

@@ -19,7 +19,6 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency "fluentd", ">=1.0.0"
   spec.add_runtime_dependency "gelf", ">= 2.0.0"
-  spec.add_development_dependency "oj", "~> 3.3.10"
   spec.add_development_dependency "test-unit"
   spec.add_development_dependency "rake"
 end

data/lib/fluent/plugin/gelf_plugin_util.rb

@@ -1,26 +1,24 @@
-require 'oj'
-require 'date'
-require 'gelf'
-
 module Fluent
   module GelfPluginUtil
+    require "gelf"
+    require "date"
 
     LEVEL_MAP = {
-      '0' => GELF::UNKNOWN, '1' => GELF::UNKNOWN, 'a' => GELF::UNKNOWN,
-      '2' => GELF::FATAL, 'c' => GELF::FATAL,
-      '3' => GELF::ERROR,
-      '4' => GELF::WARN, 'w' => GELF::WARN,
-      '5' => GELF::INFO, 'n' => GELF::INFO,
-      '6' => GELF::INFO, 'i' => GELF::INFO,
-      '7' => GELF::DEBUG, 'd' => GELF::DEBUG,
-      'e' => GELF::ERROR
+      "0" => GELF::UNKNOWN, "1" => GELF::UNKNOWN, "a" => GELF::UNKNOWN,
+      "2" => GELF::FATAL, "c" => GELF::FATAL,
+      "3" => GELF::ERROR,
+      "4" => GELF::WARN, "w" => GELF::WARN,
+      "5" => GELF::INFO, "n" => GELF::INFO,
+      "6" => GELF::INFO, "i" => GELF::INFO,
+      "7" => GELF::DEBUG, "d" => GELF::DEBUG,
+      "e" => GELF::ERROR  # assuming 'e' stands typically for 'error'
     }.freeze
 
     def make_gelfentry(tag, time, record, conf = {})
-      gelfentry = {"_fluentd_tag" => tag, "timestamp" => calculate_timestamp(time)}
+      gelfentry = {'_fluentd_tag' => tag, 'timestamp' => calculate_timestamp(time)}
 
-      record.each do |k, v|
-        process_record_entry(k, v, conf, gelfentry)
+      record.each_pair do |k, v|
+        gelfentry.merge!(process_record_entry(k, v, conf, gelfentry))
       end
 
       ensure_short_message(gelfentry)
@@ -40,29 +38,43 @@ module Fluent
     def process_record_entry(k, v, conf, gelfentry)
       case k
       when 'host', 'hostname'
-        gelfentry['host'] = conf[:use_record_host] ? v : (gelfentry['_host'] = v)
+        return {'host' => (conf[:use_record_host] ? v : gelfentry['_host'] = v)}
       when 'timestamp', 'time'
-        gelfentry['timestamp'] = parse_timestamp(v)
+        { 'timestamp' => parse_timestamp(v) }
       when 'level'
-        gelfentry['level'] = LEVEL_MAP[v.to_s.downcase[0]] || GELF::UNKNOWN
+        {'level' => LEVEL_MAP[v.to_s.downcase[0]] || (v.to_s.length >= 2 && v.to_s.downcase[1] != "r" ? GELF::UNKNOWN : v)}
       when 'msec'
-        gelfentry['timestamp'] = conf[:add_msec_time] ? "#{time.to_s}.#{v}".to_f : (gelfentry['_msec'] = v)
+        conf[:add_msec_time] ? {'timestamp' => "#{time.to_s}.#{v}".to_f} : {'_msec' => v}
       when 'short_message', 'version', 'full_message', 'facility', 'file', 'line'
-        gelfentry[k] = v
+        {k => v}
       else
-        gelfentry[k.start_with?('_') ? k : "_#{k}"] = v
+        {k.start_with?('_') ? k : "_#{k}" => v}
       end
     end
 
     def parse_timestamp(v)
-      return v if v.is_a?(Integer) || v.is_a?(Float)
-
-      DateTime.parse(v).strftime("%Q").to_f / 1_000 rescue v
+      if v.is_a?(Integer) || v.is_a?(Float)
+        v
+      else
+        begin
+          (DateTime.parse(v).strftime("%Q").to_f / 1_000).round(3)
+        rescue ArgumentError
+          v
+        end
+      end
     end
 
     def ensure_short_message(gelfentry)
-      default_key = ['_message', '_msg', '_log', '_record'].find { |key| gelfentry[key]&.strip&.empty? == false }
-      gelfentry['short_message'] = default_key ? gelfentry.delete(default_key) : '(no message)'
+      return if gelfentry['short_message'] && !gelfentry['short_message'].to_s.strip.empty?
+
+      ['_message', '_msg', '_log', '_record'].each do |key|
+        if gelfentry[key] && !gelfentry[key].to_s.strip.empty?
+          gelfentry['short_message'] = gelfentry.delete(key)
+          return
+        end
+      end
+
+      gelfentry['short_message'] = '(no message)' unless gelfentry['short_message']
     end
   end
 end

data/lib/fluent-plugin-gelf-best/version.rb

@@ -1,3 +1,3 @@
 module FluentPluginGelfBest
-  VERSION = "1.3.2"
+  VERSION = "1.3.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-gelf-best
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.3
 platform: ruby
 authors:
 - Alex Yamauchi
@@ -40,20 +40,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 2.0.0
-- !ruby/object:Gem::Dependency
-  name: oj
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.3.10
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.3.10
 - !ruby/object:Gem::Dependency
   name: test-unit
   requirement: !ruby/object:Gem::Requirement