logstash-logger 0.25.0 → 0.25.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5e1ac7a22d1e29a706c724c69e31f6d8481af909
-  data.tar.gz: 87a7b265d354455021af800ae4c22bf28ee8e3cf
+  metadata.gz: '02946cc68ecd472a33ee4e20c8b0f9676a35c31e'
+  data.tar.gz: 22ebacf159993cebd61ec2f2069c5d35ccb5949e
 SHA512:
-  metadata.gz: 9ed6a184198bde25c4a0fdbe0e4b563642e4b62ac343ae9996b07511d1b9295d37d7a5f38bcf950790b92ee1bfc1ababee32631db1f0bfcf5032f092bfd9128c
-  data.tar.gz: cac8e3fd8a892e38b1f1a01e201bbeb51bad6cb2669adbd5c317b1cedbcf208c60cdd89445e18e16b38e1576473fac104d72f2aa0ead8113455c48f8152e31bf
+  metadata.gz: d19465ebd83bb4a0710eac75face6c11287c5737fc8683e79e5b725cd81933082d123d0bcf220ab49b267c693b98f6bffc5300c7c6ceae7a0ad7b350654c3fe6
+  data.tar.gz: f1be1a3735cd768d7c53b5d9cdddb34a3fde4e76d0cc6b770f731a70cd5129d1278bd21017634c9af25c99593d7f006d3d9ed2646a4faf7c84e8e454b85a00d8

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.25.1
+
+- Skip message truncating when there is no message field. [#130](https://github.com/dwbutler/logstash-logger/pull/130)
+
 ## 0.25.0
 
 - Drops support for Rails 3.2, MRI Ruby < 2.2, and JRuby 1.7, since these have been EOL'ed.

data/README.md

@@ -817,6 +817,7 @@ logger = LogStashLogger.new('localhost', 5228, :tcp)
 * [Bin Lan](https://github.com/lanxx019)
 * [Joao Fernandes](https://github.com/jcmfernandes)
 * [CoolElvis](https://github.com/coolelvis)
+* [Sergey Pyankov](https://github.com/esergion)
 
 ## Contributing
 

data/lib/logstash-logger/formatter/base.rb

@@ -57,7 +57,7 @@ module LogStashLogger
           event.timestamp = event.timestamp.iso8601(3)
         end
 
-        if LogStashLogger.configuration.max_message_size
+        if LogStashLogger.configuration.max_message_size && event['message']
           event['message'.freeze] = event['message'.freeze].byteslice(0, LogStashLogger.configuration.max_message_size)
         end
 

data/lib/logstash-logger/version.rb

@@ -1,3 +1,3 @@
 module LogStashLogger
-  VERSION = "0.25.0"
+  VERSION = "0.25.1"
 end

data/spec/formatter/base_spec.rb

@@ -68,5 +68,36 @@ describe LogStashLogger::Formatter::Base do
         expect(event.timestamp).to eq(time.iso8601(3))
       end
     end
+
+    describe "long messages" do
+
+      context "message field is present" do
+        let(:message) { long_message }
+
+        it "truncates long messages when max_message_size is set" do
+          LogStashLogger.configure do |config|
+            config.max_message_size = 2000
+          end
+
+          expect(event['message'].size).to eq(2000)
+        end
+      end
+
+      context "event without message field" do
+        let(:message) do
+          { 'test_field' => 'test', 'foo' => 'bar' }
+        end
+
+        it "still works" do
+          LogStashLogger.configure do |config|
+            config.max_message_size = 2000
+          end
+
+          expect(event['message']).to eq(nil)
+          expect(event['test_field']).to eq('test')
+          expect(event['foo']).to eq('bar')
+        end
+      end
+    end
   end
 end

data/spec/spec_helper.rb

@@ -1,4 +1,5 @@
 require 'pry'
+require 'securerandom'
 
 require 'simplecov'
 SimpleCov.start
@@ -102,6 +103,7 @@ RSpec.shared_context 'formatter' do
   let(:time) { Time.now }
   let(:progname) { "ruby" }
   let(:message) { "foo" }
+  let(:long_message) { SecureRandom.hex(65537) }
   let(:hostname) { Socket.gethostname }
   let(:formatted_message) do
     subject.call(severity, time, progname, message)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-logger
 version: !ruby/object:Gem::Version
-  version: 0.25.0
+  version: 0.25.1
 platform: ruby
 authors:
 - David Butler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-13 00:00:00.000000000 Z
+date: 2017-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logstash-event