logstash-output-file 4.2.3 → 4.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78aedcb63cb389b41d17503f87635ec447cbc269f18baad841bfea2c408a469b
-  data.tar.gz: e70110d9d07786b566094f9b5012d2ff39eaba76312555be287c11f1e0de16bf
+  metadata.gz: 8b77d049c54896ab49beffdcfdc6c3d6263319550291b8e3e9bb7adbf693e132
+  data.tar.gz: 6c5149e2312badb7e33779600d0c65d43e0d1218ae0a7c0290f26878b706a71b
 SHA512:
-  metadata.gz: 43b2c79e85f0a59559bd48b762a1cf997f3037c65fb4e650d7d9e83cc4d4f7bc86e64a958c0ecb3568ef0b8bd519999573e68015ff6cb2d0c4f25bc00845ecc9
-  data.tar.gz: 57bfc0a2394ecdad15cd43b1777fff7c4285afa12edc833f48d3a1a0425afe89cc20f98f0be4b6a4bb64b1295d2be45a6bbf8127d602123c5b730973595e0b94
+  metadata.gz: 9040baa58fe70550583bdf6c745ca9f40a7317b660e961626ec1024f000850b3c8bb5d895d2ed12c95a85c66aadf889126c78dd9428bb35b63c7b591c3292678
+  data.tar.gz: d30d10c42110e1da662eda8a87ecc0e3d769502f2ea2e9b14f42e1cb3d82911f139f9e21c56a79ca2f9c2cae13722e09b8445931bc93e3b6dd0c12d0426fbebd

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 4.2.4
+  - Fix a bug where flush interval was being called for each event when enabled #67
+
 ## 4.2.3
   - Docs: Set the default_codec doc attribute.
 

data/lib/logstash/outputs/file.rb

@@ -18,7 +18,7 @@ require "zlib"
 # }
 class LogStash::Outputs::File < LogStash::Outputs::Base
   concurrency :shared
-  
+
   FIELD_REF = /%\{[^}]+\}/
 
   config_name "file"
@@ -48,7 +48,7 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
   # into this file and inside the defined path.
   config :filename_failure, :validate => :string, :default => '_filepath_failures'
 
-  # If the configured file is deleted, but an event is handled by the plugin, 
+  # If the configured file is deleted, but an event is handled by the plugin,
   # the plugin will recreate the file. Default => true
   config :create_if_deleted, :validate => :boolean, :default => true
 
@@ -82,7 +82,7 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
 
     @files = {}
     @io_mutex = Mutex.new
-    
+
     @path = File.expand_path(path)
 
     validate_path
@@ -141,18 +141,18 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
           # append to the file
           chunks.each {|chunk| fd.write(chunk) }
         end
-        fd.flush
+        flush(fd)
       end
-      
+
       close_stale_files
-    end   
+    end
   end # def receive
 
   public
   def close
     @io_mutex.synchronize do
       @logger.debug("Close: closing files")
-      
+
       @files.each do |path, fd|
         begin
           fd.close
@@ -180,8 +180,8 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
       file_output_path = @failure_path
     end
     @logger.debug("File, writing event to file.", :filename => file_output_path)
-    
-    file_output_path    
+
+    file_output_path
   end
 
   private
@@ -219,7 +219,7 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
       @logger.debug("Flushing file", :path => path, :fd => fd)
       fd.flush
     end
-    
+
     @last_flush_cycle = Time.now
   end
 
@@ -268,7 +268,7 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
     end
 
     @logger.info("Opening file", :path => path)
-    
+
     dir = File.dirname(path)
     if !Dir.exist?(dir)
       @logger.info("Creating directory", :directory => dir)
@@ -278,7 +278,7 @@ class LogStash::Outputs::File < LogStash::Outputs::Base
         FileUtils.mkdir_p(dir)
       end
     end
-    
+
     # work around a bug opening fifos (bug JRUBY-6280)
     stat = File.stat(path) rescue nil
     if stat && stat.ftype == "fifo" && LogStash::Environment.jruby?

data/logstash-output-file.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-output-file'
-  s.version         = '4.2.3'
+  s.version         = '4.2.4'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Writes events to files on disk"
   s.description     = "This gem is a Logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/logstash-plugin install gemname. This gem is not a stand-alone program"

data/spec/outputs/file_spec.rb

@@ -119,7 +119,8 @@ describe LogStash::Outputs::File do
         { 
           "write_behavior" => "overwrite",
           "path" => tmp,
-          "codec" => LogStash::Codecs::JSONLines.new
+          "codec" => LogStash::Codecs::JSONLines.new,
+          "flush_interval" => 0
         }
       }
       let(:output) { LogStash::Outputs::File.new(config) }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-file
 version: !ruby/object:Gem::Version
-  version: 4.2.3
+  version: 4.2.4
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-04-06 00:00:00.000000000 Z
+date: 2018-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -140,7 +140,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project:
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key:
 specification_version: 4
 summary: Writes events to files on disk