fluent-plugin-gelf-best 1.3.0 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b3d0be1e13c195cc69d517d9678fd9ed15128700a203f6fcded95a8acf26ffa5
-  data.tar.gz: b5f48294ede888f480c827fcfeb635ae0769631c8e77b0329773927ca64a0aef
+  metadata.gz: dc3a134e34dfdc0c22a93f90ca53b655bbb9133f787726adb478ffdb6b74bba3
+  data.tar.gz: f65ca691ac9b2e7046878b1b8b6315a21c6a3d575408ece21aff885c0f3c3add
 SHA512:
-  metadata.gz: 1d0a6549e8a686fca58906ca27fd153e7e6d84cac711c59a7e0312f039ad591815567d08d55724d1a88c1d7ef3ae3acef214a652fae50aac09a3fc6145aae24d
-  data.tar.gz: 60bebae56b82025efa2f141be0842d1fe966d47d0d633fbe2e18bce7dbb7718635425150ae4bce2c69713b6a6b814efd6a63af508599ea16b293c2086d74c7dc
+  metadata.gz: f2adf1721b08d85f7a191d7121eb0a0190aee674f70159c828a475a4f717a90594307e116b1a88212d0095c9b546d48134c596e7fb186f6dba9cc0476905e616
+  data.tar.gz: 7bca600938707cda06ce79d798d11dd3dd3756ee7a0a3016b2770aa9a63d1133f0a7bcb745d8e14699d6503099cced2d6ff2a63b008731e8aa8a4fc3b6a652a4

data/lib/fluent/plugin/gelf_plugin_util.rb

@@ -1,39 +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 get_internal_json_out(record, key)
-      json_data = parse_json_field(record[key])
-      return record unless json_data
-
-      record.delete(key)
-      merge_record_with_json(record, json_data)
-    rescue Oj::ParseError, EncodingError
-      # Return original record if JSON parsing fails
-      record
-    end
-
     def make_gelfentry(tag, time, record, conf = {})
-      record = get_internal_json_out(record, "message")
-      record = get_internal_json_out(record, "log")
-      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)
@@ -42,13 +27,6 @@ module Fluent
 
     private
 
-    def parse_json_field(json_field)
-      return nil unless json_field
-      json = Oj.load(json_field.strip)
-      return nil unless json.is_a?(Hash)
-      json
-    end
-
     def calculate_timestamp(time)
       if defined?(Fluent::EventTime) && time.is_a?(Fluent::EventTime)
         time.sec + (time.nsec.to_f / 1_000_000_000).round(3)
@@ -60,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.0"
+  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.0
+  version: 1.3.3
 platform: ruby
 authors:
 - Alex Yamauchi