fluent-plugin-gelf-best 1.2.0 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d561e8e54966fefedb1bfcc2ba6a629d334d3514ed378ea3fe982903b9e6b098
-  data.tar.gz: fc6f78f7a69a1b01d7046ea68c1e2334b9a8149a2c2436b938348303e318ac61
+  metadata.gz: 7327c66aa6fddfe60ed1448e76a59ecd906b458957758b748ee3162aefa85d0f
+  data.tar.gz: 2f5b1820ab88ce8665288b2e218e8be53aa274d5047e32cde33faaa5dad967c6
 SHA512:
-  metadata.gz: 5dd171597df6672a52862682cbc7735998663cf29224f4c0059baae043a09f70d01ad9420875b0131779e817941a9506afae19d03ec8b91cc4591021b36656e9
-  data.tar.gz: e63ccdd497c1fbf8bab67ed982b9e86e4f24eab987956151671c348ed19848de276603115aaf7444ce9b3ef88b4812dc0d20e84660325b266a6f821e84f63389
+  metadata.gz: 2735ce20d4dc4405d137405a30ec2c52dc83343fae9fdb7312f35e1b671546bd95ce0ab1c93648deaa1e0175fcedb79df739596aaedb27e767135695169a1ec5
+  data.tar.gz: 4add4d31fc40b78c7ff70450b556749e755f0fa6072e62a24160c4afa8ae63f09f4a46c2e2a7d26e876bb9379b65264787da37a603aa807e5dd0489c092416af

data/fluent-plugin-gelf-best.gemspec

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

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

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-gelf-best
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.2
 platform: ruby
 authors:
 - Alex Yamauchi
@@ -40,6 +40,20 @@ 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