fluent-plugin-splunk-hec 1.1.2 → 1.2.0

data/lib/fluent/plugin/out_splunk_hec/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Fluent::Plugin::SplunkHecOutput::VERSION = File.read(
   File.expand_path('../../../../VERSION', File.dirname(__FILE__))
 ).chomp.strip

data/lib/fluent/plugin/out_splunk_ingest_api.rb

@@ -0,0 +1,109 @@
+# frozen_string_literal: true
+
+require 'fluent/plugin/out_splunk'
+require 'openid_connect'
+require 'rack/oauth2'
+require 'multi_json'
+
+module Fluent::Plugin
+  class SplunkIngestApiOutput < SplunkOutput
+    Fluent::Plugin.register_output('splunk_ingest_api', self)
+
+    desc 'Service Client Identifier'
+    config_param :service_client_identifier, :string, default: nil
+
+    desc 'Service Client Secret Key'
+    config_param :service_client_secret_key, :string, default: nil
+
+    desc 'Token Endpoint'
+    config_param :token_endpoint, :string, default: '/system/identity/v1/token'
+
+    desc 'Ingest Api Hostname'
+    config_param :ingest_api_host, :string, default: 'api.splunkbeta.com'
+
+    desc 'Ingest API Tenant Name'
+    config_param :ingest_api_tenant, :string
+
+    desc 'Ingest API Events Endpoint'
+    config_param :ingest_api_events_endpoint, :string, default: '/ingest/v1beta2/events'
+
+    desc 'Debug the HTTP transport'
+    config_param :debug_http, :bool, default: false
+
+    def prefer_buffer_processing
+      true
+    end
+
+    def configure(conf)
+      super
+    end
+
+    def construct_api
+      uri = "https://#{@ingest_api_host}/#{@ingest_api_tenant}#{@ingest_api_events_endpoint}"
+      URI(uri)
+    rescue StandardError
+      raise Fluent::ConfigError, "URI #{uri} is invalid"
+    end
+
+    def format(tag, time, record)
+      format_event(tag, time, record)
+    end
+
+    def format_event(tag, time, record)
+      event = prepare_event_payload(tag, time, record)
+      # Unsure how to drop a record. So append the empty string
+      if event[:body].nil? || event[:body].strip.empty?
+        ''
+      else
+        MultiJson.dump(event) + ','
+      end
+    end
+
+    def prepare_event_payload(tag, time, record)
+      payload = super(tag, time, record)
+      payload[:attributes] = payload.delete(:fields) || {}
+      payload[:attributes][:index] = payload.delete(:index) if payload[:index]
+      payload[:body] = payload.delete(:event)
+      payload.delete(:time)
+      payload[:timestamp] = (time.to_f * 1000).to_i
+      payload[:nanos] = time.nsec / 100_000
+
+      payload
+    end
+
+    def process_response(response, request_body)
+      super
+      if response.code.to_s == '401'
+        @conn = new_connection
+        raise 'Auth Error recived. New token has been fetched.'
+      elsif response.code.to_s == '429'
+        raise "Throttle error from server. #{response.body}"
+      elsif /INVALID_DATA/.match?(response.body)
+        log.error "#{self.class}: POST Body #{request_body}"
+      end
+    end
+
+    def new_connection
+      Rack::OAuth2.debugging = true if @debug_http
+      client = OpenIDConnect::Client.new(
+        token_endpoint: @token_endpoint,
+        identifier: @service_client_identifier,
+        secret: @service_client_secret_key,
+        redirect_uri: 'http://localhost:8080/', # Not used
+        host: @ingest_api_host,
+        scheme: 'https'
+      )
+
+      client.access_token!(client_auth_method: 'other')
+    end
+
+    def write_to_splunk(chunk)
+      log.trace "#{self.class}: In write() with #{chunk.size_of_events} records and #{chunk.bytesize} bytes "
+      # ingest API is an array of json objects
+      body = "[#{chunk.read.chomp(',')}]"
+      @conn ||= new_connection
+      response = @conn.post("https://#{@ingest_api_host}/#{@ingest_api_tenant}#{@ingest_api_events_endpoint}", body: body)
+      process_response(response, body)
+    end
+  end
+end

data/test/fluent/plugin/out_splunk_hec_test.rb

@@ -1,92 +1,103 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 
 describe Fluent::Plugin::SplunkHecOutput do
   include Fluent::Test::Helpers
   include PluginTestHelper
 
   before { Fluent::Test.setup } # setup router and others
-    
+
   it { expect(::Fluent::Plugin::SplunkHecOutput::VERSION).wont_be_nil }
 
-  describe "config param tests" do
-      it "should require https protocol" do
-        expect(create_output_driver('hec_host protocol').instance.protocol).must_equal :https
-      end
-      it "should require hec_host" do
-        expect(create_output_driver('hec_host hec_host').instance.hec_host).must_equal "hec_host"
-      end
-      it "should require hec_port" do
-        expect(create_output_driver('hec_host hec_port').instance.hec_port).must_equal 8088
-      end
-      it "should require hec_token" do
-        expect(create_output_driver('hec_host hec_token').instance.hec_token).must_equal "some-token"
-      end
-      it "should define client_cert as nil initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.client_cert)
-      end
-      it "should define client_key as nil (string) initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.client_key)
-        expect(create_output_driver('hec_host hec_token').instance.client_key).is_a? String
-      end
-      it "should define ca_file as nil (string) initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.ca_file)
-        expect(create_output_driver('hec_host hec_token').instance.ca_file).is_a? String
-      end
-      it "should define ca_path as nil (string) initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.ca_path)
-        expect(create_output_driver('hec_host hec_token').instance.ca_path).is_a? String
-      end
-      it "should define ssl_ciphers as nil (array) initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.ssl_ciphers)
-        expect(create_output_driver('hec_host hec_token').instance.ssl_ciphers).is_a? Array
-      end
-      it "should not allow an insecure ssl connection" do
-        expect(create_output_driver('hec_host hec_token').instance.insecure_ssl).must_equal false
-      end
-      it "should allow both event (default) and metric to be sent to splunk" do
-        expect(create_output_driver('hec_host hec_token').instance.data_type).must_equal :event
-        expect(create_output_driver('hec_host hec_token').instance.data_type = :metric).must_equal :metric
-      end
-      it "should define Splunk index to index (string) as nil initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.index)
-        expect(create_output_driver('hec_host hec_token').instance.index).is_a? String
-      end
-      it "should define field names to include Splunk index_key as nil (string) initially" do
-        assert_nil(create_output_driver('hec_host hec_token').instance.index_key)
-        expect(create_output_driver('hec_host hec_token').instance.index_key).is_a? String
-      end
+  describe 'config param tests' do
+    it 'should require https protocol' do
+      expect(create_hec_output_driver('hec_host protocol').instance.protocol).must_equal :https
+    end
+    it 'should require hec_host' do
+      expect(create_hec_output_driver('hec_host hec_host').instance.hec_host).must_equal 'hec_host'
+    end
+    it 'should require hec_port' do
+      expect(create_hec_output_driver('hec_host hec_port').instance.hec_port).must_equal 8088
+    end
+    it 'should require hec_token' do
+      expect(create_hec_output_driver('hec_host hec_token').instance.hec_token).must_equal 'some-token'
+    end
+    it 'should define client_cert as nil initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.client_cert)
+    end
+    it 'should define client_key as nil (string) initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.client_key)
+      expect(create_hec_output_driver('hec_host hec_token').instance.client_key).is_a? String
+    end
+    it 'should define ca_file as nil (string) initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.ca_file)
+      expect(create_hec_output_driver('hec_host hec_token').instance.ca_file).is_a? String
+    end
+    it 'should define ca_path as nil (string) initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.ca_path)
+      expect(create_hec_output_driver('hec_host hec_token').instance.ca_path).is_a? String
+    end
+    it 'should define ssl_ciphers as nil (array) initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.ssl_ciphers)
+      expect(create_hec_output_driver('hec_host hec_token').instance.ssl_ciphers).is_a? Array
+    end
+    it 'should not allow an insecure ssl connection' do
+      expect(create_hec_output_driver('hec_host hec_token').instance.insecure_ssl).must_equal false
+    end
+    it 'should allow both event (default) and metric to be sent to splunk' do
+      expect(create_hec_output_driver('hec_host hec_token').instance.data_type).must_equal :event
+      expect(create_hec_output_driver('hec_host hec_token').instance.data_type = :metric).must_equal :metric
+    end
+    it 'should define Splunk index to index (string) as nil initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.index)
+      expect(create_hec_output_driver('hec_host hec_token').instance.index).is_a? String
+    end
+    it 'should define field names to include Splunk index_key as nil (string) initially' do
+      assert_nil(create_hec_output_driver('hec_host hec_token').instance.index_key)
+      expect(create_hec_output_driver('hec_host hec_token').instance.index_key).is_a? String
+    end
   end
 
-  describe "hec_host validation" do
-    describe "invalid host" do
-      it "should require hec_host" do
-	      expect{ create_output_driver }.must_raise Fluent::ConfigError
+  describe 'hec_host validation' do
+    describe 'invalid host' do
+      it 'should require hec_host' do
+        expect { create_hec_output_driver }.must_raise Fluent::ConfigError
       end
 
-      it { expect{ create_output_driver('hec_host %bad-host%') }.must_raise Fluent::ConfigError }
+      it { expect { create_hec_output_driver('hec_host %bad-host%') }.must_raise Fluent::ConfigError }
     end
 
-    describe "good host" do
+    describe 'good host' do
       it {
-	      expect(create_output_driver('hec_host splunk.com').instance.hec_host).must_equal "splunk.com"
+        expect(create_hec_output_driver('hec_host splunk.com').instance.hec_host).must_equal 'splunk.com'
       }
     end
   end
 
-  it "should send request to Splunk" do
-    req = verify_sent_events { |batch|
+  it 'should send request to Splunk' do
+    req = verify_sent_events do |batch|
       expect(batch.size).must_equal 2
-    }
+    end
     expect(req).must_be_requested times: 1
   end
 
-  it "should use string for event time, and the value of the string should be a float" do
-    verify_sent_events { |batch|
+  it 'should use string for event time, and the value of the string should be a float' do
+    verify_sent_events do |batch|
       batch.each do |item|
         expect(item['time']).must_be_instance_of String
         expect(item['time']).must_match /^\d+\.\d+$/
       end
-    }
+    end
+  end
+
+  # it "should contain splunk event time field via fluentd, as nil" do
+  #   expect(create_hec_output_driver('hec_host splunk.com').instance.time_key).must_equal nil
+  # end
+  #
+  it 'should contain splunk event time field via fluentd, as nil' do
+    test_driver = create_hec_output_driver('hec_host splunk.com')
+    assert_nil(test_driver.instance.time_key)
   end
 
   # it "should contain splunk event time field via fluentd, as nil" do
@@ -99,236 +110,236 @@ describe Fluent::Plugin::SplunkHecOutput do
   end
 
   it "should use host machine's hostname for event host by default" do
-    verify_sent_events { |batch|
+    verify_sent_events do |batch|
       batch.each do |item|
-	      expect(item['host']).must_equal Socket.gethostname
+        expect(item['host']).must_equal Socket.gethostname
       end
-    }
+    end
   end
 
   %w[index source sourcetype].each do |field|
     it "should not set #{field} by default" do
-      verify_sent_events { |batch|
-	    batch.each do |item|
-        expect(item).wont_include field
-	    end
-      }
+      verify_sent_events do |batch|
+        batch.each do |item|
+          expect(item).wont_include field
+        end
+      end
     end
   end
 
-  it "should support ${tag}" do
-    verify_sent_events(<<~CONF) { |batch|
-    index ${tag}
-    host ${tag}
-    source ${tag}
-    sourcetype ${tag}
+  it 'should support ${tag}' do
+    verify_sent_events(<<~CONF) do |batch|
+      index ${tag}
+      host ${tag}
+      source ${tag}
+      sourcetype ${tag}
     CONF
       batch.each do |item|
-	    %w[index host source sourcetype].each { |field|
-	      expect(%w[tag.event1 tag.event2]).must_include item[field]
-	    }
+        %w[index host source sourcetype].each do |field|
+          expect(%w[tag.event1 tag.event2]).must_include item[field]
+        end
       end
-    }
+    end
   end
 
-  it "should support *_key" do
-    verify_sent_events(<<~CONF) { |batch|
+  it 'should support *_key' do
+    verify_sent_events(<<~CONF) do |batch|
       index_key      level
       host_key       from
       source_key     file
       sourcetype_key agent.name
     CONF
-      batch.each { |item|
+      batch.each do |item|
         expect(item['index']).must_equal 'info'
         expect(item['host']).must_equal 'my_machine'
         expect(item['source']).must_equal 'cool.log'
         expect(item['sourcetype']).must_equal 'test'
 
-	JSON.load(item['event']).tap do |event|
-	  %w[level from file].each { |field| expect(event).wont_include field }
-	  expect(event['agent']).wont_include 'name'
-	end
-      }
-    }
+        JSON.load(item['event']).tap do |event|
+          %w[level from file].each { |field| expect(event).wont_include field }
+          expect(event['agent']).wont_include 'name'
+        end
+      end
+    end
   end
 
-  it "should remove nil fields." do
-    verify_sent_events(<<~CONF) { |batch|
+  it 'should remove nil fileds.' do
+    verify_sent_events(<<~CONF) do |batch|
       index_key      nonexist
       host_key       nonexist
       source_key     nonexist
       sourcetype_key nonexist
     CONF
-      batch.each { |item|
-	expect(item).wont_be :has_key?, 'index'
-	expect(item).wont_be :has_key?, 'host'
-	expect(item).wont_be :has_key?, 'source'
-	expect(item).wont_be :has_key?, 'sourcetype'
-      }
-    }
+      batch.each do |item|
+        expect(item).wont_be :has_key?, 'index'
+        expect(item).wont_be :has_key?, 'host'
+        expect(item).wont_be :has_key?, 'source'
+        expect(item).wont_be :has_key?, 'sourcetype'
+      end
+    end
   end
 
   describe 'formatter' do
-    it "should support replace the default json formatter" do
-      verify_sent_events(<<~CONF) { |batch|
-	<format>
-	  @type single_value
-	  message_key log
-	  add_newline false
-	</format>
+    it 'should support replace the default json formater' do
+      verify_sent_events(<<~CONF) do |batch|
+        <format>
+          @type single_value
+          message_key log
+          add_newline false
+        </format>
       CONF
-	batch.map { |item| item['event'] }
-	     .each { |event| expect(event).must_equal "everything is good" }
-      }
+        batch.map { |item| item['event'] }
+             .each { |event| expect(event).must_equal 'everything is good' }
+      end
     end
 
-    it "should support multiple formatters" do
-      verify_sent_events(<<~CONF) { |batch|
-	source ${tag}
-	<format tag.event1>
-	  @type single_value
-	  message_key log
-	  add_newline false
-	</format>
+    it 'should support multiple formatters' do
+      verify_sent_events(<<~CONF) do |batch|
+        source ${tag}
+        <format tag.event1>
+          @type single_value
+          message_key log
+          add_newline false
+        </format>
       CONF
-	expect(batch.find { |item| item['source'] == 'tag.event1' }['event']).must_equal "everything is good"
-	expect(batch.find { |item| item['source'] == 'tag.event2' }['event']).must_be_instance_of Hash
-      }
+        expect(batch.find { |item| item['source'] == 'tag.event1' }['event']).must_equal 'everything is good'
+        expect(batch.find { |item| item['source'] == 'tag.event2' }['event']).must_be_instance_of Hash
+      end
     end
   end
 
-  it "should support fields for indexed field extraction" do
-    verify_sent_events(<<~CONF) { |batch|
-    <fields>
-      from
-      logLevel level
-      nonexist
-    </fields>
+  it 'should support fields for indexed field extraction' do
+    verify_sent_events(<<~CONF) do |batch|
+      <fields>
+        from
+        logLevel level
+        nonexist
+      </fields>
     CONF
       batch.each do |item|
-	JSON.load(item['event']).tap { |event|
-	  expect(event).wont_include 'from'
-	  expect(event).wont_include 'level'
-	}
-
-	expect(item['fields']['from']).must_equal 'my_machine'
-	expect(item['fields']['logLevel']).must_equal 'info'
-	expect(item['fields']).wont_be :has_key?, 'nonexist'
+        JSON.load(item['event']).tap do |event|
+          expect(event).wont_include 'from'
+          expect(event).wont_include 'level'
+        end
+
+        expect(item['fields']['from']).must_equal 'my_machine'
+        expect(item['fields']['logLevel']).must_equal 'info'
+        expect(item['fields']).wont_be :has_key?, 'nonexist'
       end
-    }
+    end
   end
 
-  describe 'metric'do
+  describe 'metric' do
     it 'should check related configs' do
       expect(
-	      create_output_driver('hec_host somehost', 'data_type metric')
+        create_hec_output_driver('hec_host somehost', 'data_type metric')
       ).wont_be_nil
 
-      expect{
-	      create_output_driver('hec_host somehost', 'data_type metric', 'metrics_from_event false')
-      }.must_raise Fluent::ConfigError
+      expect do
+        create_hec_output_driver('hec_host somehost', 'data_type metric', 'metrics_from_event false')
+      end.must_raise Fluent::ConfigError
 
-      expect{
-	      create_output_driver('hec_host somehost', 'data_type metric', 'metric_name_key x')
-      }.must_raise Fluent::ConfigError
+      expect do
+        create_hec_output_driver('hec_host somehost', 'data_type metric', 'metric_name_key x')
+      end.must_raise Fluent::ConfigError
 
       expect(
-	      create_output_driver('hec_host somehost', 'data_type metric', 'metric_name_key x', 'metric_value_key y')
+        create_hec_output_driver('hec_host somehost', 'data_type metric', 'metric_name_key x', 'metric_value_key y')
       ).wont_be_nil
     end
 
     it 'should have "metric" as event, and have proper fields' do
-      verify_sent_events(<<~CONF) { |batch|
-      data_type metric
-      metric_name_key from
-      metric_value_key value
+      verify_sent_events(<<~CONF) do |batch|
+        data_type metric
+        metric_name_key from
+        metric_value_key value
       CONF
-      batch.each do |item|
-        expect(item['event']).must_equal 'metric'
-        expect(item['fields']['metric_name']).must_equal 'my_machine'
-        expect(item['fields']['_value']).must_equal 100
-        expect(item['fields']['log']).must_equal 'everything is good'
-        expect(item['fields']['level']).must_equal 'info'
-        expect(item['fields']['file']).must_equal 'cool.log'
-	    end
-      }
+        batch.each do |item|
+          expect(item['event']).must_equal 'metric'
+          expect(item['fields']['metric_name']).must_equal 'my_machine'
+          expect(item['fields']['_value']).must_equal 100
+          expect(item['fields']['log']).must_equal 'everything is good'
+          expect(item['fields']['level']).must_equal 'info'
+          expect(item['fields']['file']).must_equal 'cool.log'
+        end
+      end
     end
 
     it 'should handle empty fields' do
-      verify_sent_events(<<~CONF) { |batch|
-      data_type metric
-      metric_name_key from
-      metric_value_key value
-      <fields>
-      </fields>
+      verify_sent_events(<<~CONF) do |batch|
+        data_type metric
+        metric_name_key from
+        metric_value_key value
+        <fields>
+        </fields>
       CONF
-      batch.each do |item|
-	  # only "metric_name" and "_value"
-	      expect(item['fields'].keys.size).must_equal 2
-	    end
-      }
+        batch.each do |item|
+          # only "metric_name" and "_value"
+          expect(item['fields'].keys.size).must_equal 2
+        end
+      end
     end
 
     it 'should handle custom fields' do
-      verify_sent_events(<<~CONF) { |batch|
-      data_type metric
-      metric_name_key from
-      metric_value_key value
-      <fields>
-        level
-        filePath file
-        username
-      </fields>
+      verify_sent_events(<<~CONF) do |batch|
+        data_type metric
+        metric_name_key from
+        metric_value_key value
+        <fields>
+          level
+          filePath file
+          username
+        </fields>
       CONF
-      batch.each do |item|
-        expect(item['fields'].keys.size).must_equal 4
-        expect(item['fields']['level']).must_equal 'info'
-        expect(item['fields']['filePath']).must_equal 'cool.log'
-        # null fields should be removed
-        expect(item['fields']).wont_be :has_key?, 'username'
-	    end
-      }
+        batch.each do |item|
+          expect(item['fields'].keys.size).must_equal 4
+          expect(item['fields']['level']).must_equal 'info'
+          expect(item['fields']['filePath']).must_equal 'cool.log'
+          # null fields should be removed
+          expect(item['fields']).wont_be :has_key?, 'username'
+        end
+      end
     end
 
     it 'should treat each key-value in event as a metric' do
       metrics = [
-        ['tag', event_time, {'cup': 0.5, 'memory': 100}],
-        ['tag', event_time, {'cup': 0.6, 'memory': 200}]
+        ['tag', event_time, { 'cup': 0.5, 'memory': 100 }],
+        ['tag', event_time, { 'cup': 0.6, 'memory': 200 }]
       ]
-      with_stub_hec(events: metrics, conf: 'data_type metric') { |batch|
-	      expect(batch.size).must_equal 4
-      }
+      with_stub_hec(events: metrics, conf: 'data_type metric') do |batch|
+        expect(batch.size).must_equal 4
+      end
     end
   end
 
   describe 'timeout params' do
-      it 'should reset unused connection after 5 seconds' do
-        expect(create_output_driver('hec_host splunk.com', 'idle_timeout 5').instance.idle_timeout).must_equal 5
-      end
+    it 'should reset unused connection after 5 seconds' do
+      expect(create_hec_output_driver('hec_host splunk.com', 'idle_timeout 5').instance.idle_timeout).must_equal 5
+    end
 
-      it 'should allow custom setting between reading chunks from the socket' do
-        expect(create_output_driver('hec_host splunk.com', 'read_timeout 5').instance.read_timeout).must_equal 5
-      end
+    it 'should allow custom setting between reading chunks from the socket' do
+      expect(create_hec_output_driver('hec_host splunk.com', 'read_timeout 5').instance.read_timeout).must_equal 5
+    end
 
-      it 'should allow custom setting a connection to be opened' do
-        expect(create_output_driver('hec_host splunk.com',  'open_timeout 5').instance.open_timeout).must_equal 5
-      end
+    it 'should allow custom setting a connection to be opened' do
+      expect(create_hec_output_driver('hec_host splunk.com', 'open_timeout 5').instance.open_timeout).must_equal 5
+    end
 
-      it 'should check default values are created correctly for timeout params' do
-        test_driver = create_output_driver('hec_host splunk.com')
-        expect(test_driver.instance.idle_timeout).must_equal 5
-        assert_nil(test_driver.instance.read_timeout)
-        assert_nil(test_driver.instance.open_timeout)
-      end
+    it 'should check default values are created correctly for timeout params' do
+      test_driver = create_hec_output_driver('hec_host splunk.com')
+      expect(test_driver.instance.idle_timeout).must_equal 5
+      assert_nil(test_driver.instance.read_timeout)
+      assert_nil(test_driver.instance.open_timeout)
     end
+  end
 
-  def with_stub_hec(events:, conf: '', &blk)
-    host = "hec.splunk.com"
-    @driver = create_output_driver("hec_host #{host}", conf)
+  def with_stub_hec(events:, conf: '')
+    host = 'hec.splunk.com'
+    @driver = create_hec_output_driver("hec_host #{host}", conf)
 
-    hec_req = stub_hec_request("https://#{host}:8088").with { |r|
-      blk.call r.body.split(/(?={)\s*(?<=})/).map { |item| JSON.load item }
-    }
+    hec_req = stub_hec_request("https://#{host}:8088").with do |r|
+      yield r.body.split(/(?={)\s*(?<=})/).map { |item| JSON.load item }
+    end
 
     @driver.run do
       events.each { |evt| @driver.feed *evt }
@@ -339,19 +350,19 @@ describe Fluent::Plugin::SplunkHecOutput do
 
   def verify_sent_events(conf = '', &blk)
     event = {
-      "log"   => "everything is good",
-      "level" => "info",
-      "from"  => "my_machine",
-      "file"  => "cool.log",
-      "value" => 100,
-      "agent" => {
-	      "name"    => "test",
-	      "version" => "1.1.0"
+      'log' => 'everything is good',
+      'level' => 'info',
+      'from' => 'my_machine',
+      'file' => 'cool.log',
+      'value' => 100,
+      'agent' => {
+        'name' => 'test',
+        'version' => '1.0.0'
       }
     }
     events = [
-      ["tag.event1", event_time, {"id" => "1st"}.merge(Marshal.load(Marshal.dump(event)))],
-      ["tag.event2", event_time, {"id" => "2nd"}.merge(Marshal.load(Marshal.dump(event)))]
+      ['tag.event1', event_time, { 'id' => '1st' }.merge(Marshal.load(Marshal.dump(event)))],
+      ['tag.event2', event_time, { 'id' => '2nd' }.merge(Marshal.load(Marshal.dump(event)))]
     ]
 
     with_stub_hec conf: conf, events: events, &blk