logstash-filter-mutate 3.5.7 → 3.5.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2e5b32e8796c3dc7709a89d2e345fbd720f4c144dc90150833e357c052473b6
-  data.tar.gz: ce9ac9d9dc6b3ea1e87e3628c6f7a3725aa4defaea47feffbbfbc966824c8113
+  metadata.gz: a35944145cf520f4dc7a0249e8d4c75e60abcf0c454545f0fb9a4a70af223bea
+  data.tar.gz: 431b8427123636a98fa9c51a4460157d6018cd594c927e95e9028048f49c2e3a
 SHA512:
-  metadata.gz: 24571cc6195435dc6edfc23cd3c4abae6c6041f77acc5749c5e944ad10f2ed71457b89bbc8c152d836855460ee3b8b0ddbb82139d4b4f1c398f2e9602005161a
-  data.tar.gz: 2be971ad1a39c3e86a6af402c3a08710db26c86fecb7e8a28878bb87ca4e1cb7f72eb7d4290f49079af78252dd3165c17928e0f5d5da7aaf069bf11beb2476f3
+  metadata.gz: 1c7c36f559f8733d38ba8d87f2017418b380c1178100fb4a7641cb5c80e5440b06e5e6aaed46b3f9c53be3a5aeaa245bb3f2fd90a90a41ddd105c3ff00046ffb
+  data.tar.gz: b8b4c405276c6714db052d3cd1a46a5286c4ef787dd9aad33b7861468e6d83410f13eea597dc41aee0a03d6e110a40ace196bb687efba0caa417114bfdeb6bb3

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 3.5.8
+  - Fix "Can't modify frozen string" error when converting boolean to `string` [#171](https://github.com/logstash-plugins/logstash-filter-mutate/pull/171) 
+  
 ## 3.5.7
   - Clarify that `split` and `join` also support strings [#164](https://github.com/logstash-plugins/logstash-filter-mutate/pull/164)
 

data/lib/logstash/filters/mutate.rb

@@ -330,7 +330,8 @@ class LogStash::Filters::Mutate < LogStash::Filters::Base
     # target encoding and only change if necessary, so calling
     # valid_encoding? is redundant
     # see https://twitter.com/jordansissel/status/444613207143903232
-    value.to_s.force_encoding(Encoding::UTF_8)
+    # use + since .to_s on nil/boolean returns a frozen string since ruby 2.7
+    (+value.to_s).force_encoding(Encoding::UTF_8)
   end
 
   def convert_boolean(value)

data/logstash-filter-mutate.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-filter-mutate'
-  s.version         = '3.5.7'
+  s.version         = '3.5.8'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Performs mutations on fields"
   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/filters/integration/multi_stage_spec.rb

@@ -34,7 +34,7 @@ describe 'LogStash::Filters::Mutate' do
 CONFIG
     end
 
-    sample("message" => "hello WORLD", "lower1" => "PPQQRRSS", "lower2" => "pppqqq") do
+    sample({"message" => "hello WORLD", "lower1" => "PPQQRRSS", "lower2" => "pppqqq"}) do
       result = results.first
       expect(result.get("bar")).to eq('world')
       expect(result.get("lower1")).to eq("ppqqrrss")

data/spec/filters/mutate_spec.rb

@@ -428,7 +428,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("unicorns" => 1234) do
+    sample({"unicorns" => 1234}) do
       expect(subject.get("unicorns")).to eq "1234"
     end
   end
@@ -561,25 +561,25 @@ describe LogStash::Filters::Mutate do
     end
 
     context 'when field is the true value' do
-      sample('field' => true) do
+      sample({'field' => true}) do
         expect(subject.get('field')).to eq(1.0)
       end
     end
 
     context 'when field is the false value' do
-      sample('field' => false) do
+      sample({'field' => false}) do
         expect(subject.get('field')).to eq(0.0)
       end
     end
 
     context 'when field is nil' do
-      sample('field' => nil) do
+      sample({'field' => nil}) do
         expect(subject.get('field')).to be_nil
       end
     end
 
     context 'when field is not set' do
-      sample('field' => nil) do
+      sample({'field' => nil}) do
         expect(subject.get('field')).to be_nil
       end
     end
@@ -634,25 +634,25 @@ describe LogStash::Filters::Mutate do
     end
 
     context 'when field is the true value' do
-      sample('field' => true) do
+      sample({'field' => true}) do
         expect(subject.get('field')).to eq(1.0)
       end
     end
 
     context 'when field is the false value' do
-      sample('field' => false) do
+      sample({'field' => false}) do
         expect(subject.get('field')).to eq(0.0)
       end
     end
 
     context 'when field is nil' do
-      sample('field' => nil) do
+      sample({'field' => nil}) do
         expect(subject.get('field')).to be_nil
       end
     end
 
     context 'when field is not set' do
-      sample('field' => nil) do
+      sample({'field' => nil}) do
         expect(subject.get('field')).to be_nil
       end
     end
@@ -666,7 +666,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("unicorns" => "Magnificient, but extinct, animals") do
+    sample({"unicorns" => "Magnificient, but extinct, animals"}) do
       expect(subject.get("unicorns")).to eq "Magnificient, and common, animals"
     end
   end
@@ -679,8 +679,8 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("unicorns" => [
-      "Magnificient extinct animals", "Other extinct ideas" ]
+    sample({"unicorns" => [
+      "Magnificient extinct animals", "Other extinct ideas" ]}
     ) do
       expect(subject.get("unicorns")).to eq [
         "Magnificient common animals",
@@ -698,7 +698,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("colors" => "One red car", "shapes" => "Four red squares") do
+    sample({"colors" => "One red car", "shapes" => "Four red squares"}) do
       expect(subject.get("colors")).to eq "One blue car"
       expect(subject.get("shapes")).to eq "Four red circles"
     end
@@ -712,7 +712,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("colors" => "red3") do
+    sample({"colors" => "red3"}) do
       expect(subject.get("colors")).to eq "redblue"
     end
   end
@@ -758,7 +758,7 @@ describe LogStash::Filters::Mutate do
       }
     CONFIG
 
-    sample("field_one" => "value", "x" => "one") do
+    sample({"field_one" => "value", "x" => "one"}) do
       expect(subject).to_not include("field_one")
       expect(subject).to include("destination")
     end
@@ -773,7 +773,7 @@ describe LogStash::Filters::Mutate do
       }
     CONFIG
 
-    sample("field_one" => "value", "x" => "one") do
+    sample({"field_one" => "value", "x" => "one"}) do
       expect(subject).to_not include("origin")
       expect(subject).to include("field_one")
     end
@@ -934,6 +934,26 @@ describe LogStash::Filters::Mutate do
     end
   end
 
+  describe "convert auto-frozen values to string" do
+    config <<-CONFIG
+      filter {
+        mutate {
+          convert => {
+            "true_field"  => "string"
+            "false_field" => "string"
+          }
+        }
+      }
+    CONFIG
+
+    sample({ "true_field" => true, "false_field" => false }) do
+      expect(subject.get("true_field")).to eq "true"
+      expect(subject.get("true_field")).to be_a(String)
+      expect(subject.get("false_field")).to eq "false"
+      expect(subject.get("false_field")).to be_a(String)
+    end
+  end
+
   #LOGSTASH-1529
   describe "gsub on a String with dynamic fields (%{}) in pattern" do
     config '
@@ -943,7 +963,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("unicorns" => "Unicorns of type blue are common", "unicorn_type" => "blue") do
+    sample({"unicorns" => "Unicorns of type blue are common", "unicorn_type" => "blue"}) do
       expect(subject.get("unicorns")).to eq "Unicorns green are common"
     end
   end
@@ -957,7 +977,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("unicorns2" => "Unicorns of type blue are common", "unicorn_color" => "blue") do
+    sample({"unicorns2" => "Unicorns of type blue are common", "unicorn_color" => "blue"}) do
       expect(subject.get("unicorns2")).to eq "Unicorns blue and green are common"
     end
   end
@@ -971,9 +991,9 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("unicorns_array" => [
+    sample({"unicorns_array" => [
         "Unicorns of type blue are found in Alaska", "Unicorns of type blue are extinct" ],
-           "color" => "blue"
+           "color" => "blue" }
     ) do
       expect(subject.get("unicorns_array")).to eq [
           "Unicorns blue and green are found in Alaska",
@@ -990,7 +1010,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => "bar") do
+    sample({"foo" => "bar"}) do
       expect(subject.get("list")).to eq ["bar"]
       expect(subject.get("foo")).to eq "bar"
     end
@@ -1004,7 +1024,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => "bar", "list" => []) do
+    sample({"foo" => "bar", "list" => []}) do
       expect(subject.get("list")).to eq ["bar"]
       expect(subject.get("foo")).to eq "bar"
     end
@@ -1018,7 +1038,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => "bar", "list" => ["baz"]) do
+    sample({"foo" => "bar", "list" => ["baz"]}) do
       expect(subject.get("list")).to eq ["baz", "bar"]
       expect(subject.get("foo")).to eq "bar"
     end
@@ -1032,7 +1052,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => ["bar"], "list" => ["baz"]) do
+    sample({"foo" => ["bar"], "list" => ["baz"]}) do
       expect(subject.get("list")).to eq ["baz", "bar"]
       expect(subject.get("foo")).to eq ["bar"]
     end
@@ -1046,7 +1066,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => [], "list" => ["baz"]) do
+    sample({"foo" => [], "list" => ["baz"]}) do
       expect(subject.get("list")).to eq ["baz"]
       expect(subject.get("foo")).to eq []
     end
@@ -1060,7 +1080,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => ["bar"], "list" => "baz") do
+    sample({"foo" => ["bar"], "list" => "baz"}) do
       expect(subject.get("list")).to eq ["baz", "bar"]
       expect(subject.get("foo")).to eq ["bar"]
     end
@@ -1074,7 +1094,7 @@ describe LogStash::Filters::Mutate do
         }
       }'
 
-    sample("foo" => "bar", "list" => "baz") do
+    sample({"foo" => "bar", "list" => "baz"}) do
       expect(subject.get("list")).to eq ["baz", "bar"]
       expect(subject.get("foo")).to eq "bar"
     end
@@ -1094,7 +1114,7 @@ describe LogStash::Filters::Mutate do
       }'
 
 
-    sample("field1" => nil, "field2" => nil, "field3" => nil, "field4" => true) do
+    sample({"field1" => nil, "field2" => nil, "field3" => nil, "field4" => true}) do
       expect(subject.get("field1")).to eq("Hello")
       expect(subject.get("field2")).to eq("Bye")
       expect(subject.get("field3")).to eq("5")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-mutate
 version: !ruby/object:Gem::Version
-  version: 3.5.7
+  version: 3.5.8
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-11 00:00:00.000000000 Z
+date: 2023-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement