logstash-filter-mutate 3.5.7 → 3.5.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2e5b32e8796c3dc7709a89d2e345fbd720f4c144dc90150833e357c052473b6
-  data.tar.gz: ce9ac9d9dc6b3ea1e87e3628c6f7a3725aa4defaea47feffbbfbc966824c8113
+  metadata.gz: 9fd4fb3b414af0266ba7a62e6ef72b00ad1b109200d8da6b54ee7910a1ba561b
+  data.tar.gz: 6368b980d3d5c1b32f139bfc007341cafff738a2db055df8c3534436c836b729
 SHA512:
-  metadata.gz: 24571cc6195435dc6edfc23cd3c4abae6c6041f77acc5749c5e944ad10f2ed71457b89bbc8c152d836855460ee3b8b0ddbb82139d4b4f1c398f2e9602005161a
-  data.tar.gz: 2be971ad1a39c3e86a6af402c3a08710db26c86fecb7e8a28878bb87ca4e1cb7f72eb7d4290f49079af78252dd3165c17928e0f5d5da7aaf069bf11beb2476f3
+  metadata.gz: 6ba63632430477006fc77d5d27e3ee14c72f4e1bc5bd9557ba886c31b95b34636fd1ed7869dfc77c3a95d02355b04f0afa1e4a4c6053b715d8769c7c5fc1895b
+  data.tar.gz: b971f7880a0a98ee89545b5775fcde5c67adb48b624c5b6885b6a70476352950c246d1577d9e133b82b1979b89f7f1500b156a27ed5f6ca3a4fa016e327767e0

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 3.5.9
+  - Fix `convert` to covert hexadecimal float notation and scientific notation string to float and integer [#175](https://github.com/logstash-plugins/logstash-filter-mutate/pull/175)
+  
+## 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

@@ -1,6 +1,7 @@
 # encoding: utf-8
 require "logstash/filters/base"
 require "logstash/namespace"
+require "bigdecimal"
 
 # The mutate filter allows you to perform general mutations on fields. You
 # can rename, replace, and modify fields in your events.
@@ -227,6 +228,10 @@ class LogStash::Filters::Mutate < LogStash::Filters::Base
         )
       end
     end
+    # fallback to old type conversion behavior prior to Logstash 8.14.0
+    @lenient_conversion = self.class.is_lenient_version?
+    # signed hex parsing support in jruby 10
+    @support_signed_hex = self.class.can_parse_signed_hex?
 
     @gsub_parsed = []
     @gsub.nil? or @gsub.each_slice(3) do |field, needle, replacement|
@@ -330,7 +335,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)
@@ -341,17 +347,44 @@ class LogStash::Filters::Mutate < LogStash::Filters::Base
   end
 
   def convert_integer(value)
+    value = value.strip.delete(',').downcase if value.is_a?(String)
+
     return 1 if value == true
     return 0 if value == false
-    return value.to_i if !value.is_a?(String)
-    value.tr(",", "").to_i
+    return value if value.is_a?(Integer)
+    return value.to_i if @lenient_conversion
+
+    if value.is_a?(String)
+      signed_float = parse_signed_hex_str(value)
+      # hex number
+      if signed_float
+        return Integer(value) if value.count(".").zero?
+        return Integer(signed_float)
+      end
+
+      # scientific notation. BigDecimal() can't parse hex string
+      return BigDecimal(value).to_i if value.include?("e")
+      # maybe a float string
+      return value.to_i
+    end
+
+    Integer(value)
   end
 
   def convert_float(value)
+    value = value.strip.delete(',').downcase if value.is_a?(String)
+
     return 1.0 if value == true
     return 0.0 if value == false
-    value = value.delete(",") if value.kind_of?(String)
-    value.to_f
+    return value if value.is_a?(Float)
+    return value.to_f if @lenient_conversion
+
+    if value.is_a?(String)
+      signed_float = parse_signed_hex_str(value)
+      return signed_float if signed_float
+    end
+
+    Float(value)
   end
 
   def convert_integer_eu(value)
@@ -373,6 +406,25 @@ class LogStash::Filters::Mutate < LogStash::Filters::Base
     value.tr(",.", ".,")
   end
 
+  # Parses a string to determine if it represents a signed hexadecimal number.
+  # If the string matches a signed hex format (eg "-0x1A"), returns the signed float value.
+  # JRuby Float() can parse signed hex string and uppercase hex string in version 10+,
+  # but not in earlier versions.
+  #
+  # @param value [String] the string to parse
+  # @return [Float, nil] the signed float value if hex, or nil if not a hex string
+  def parse_signed_hex_str(value)
+    return Float(value) if @support_signed_hex
+
+    if value.match?(/^[+-]?0x/i)
+      sign = value.start_with?('-') ? -1 : 1
+      unsigned = value.sub(/^[+-]/, '')
+      return sign * Float(unsigned)
+    end
+
+    nil
+  end
+
   def gsub(event)
     @gsub_parsed.each do |config|
       field = config[:field]
@@ -541,4 +593,14 @@ class LogStash::Filters::Mutate < LogStash::Filters::Base
       event.set(dest_field,LogStash::Util.deep_clone(original))
     end
   end
+
+  def self.is_lenient_version?
+    Gem::Version.new(LOGSTASH_VERSION) < Gem::Version.new("8.14.0")
+  end
+
+  def self.can_parse_signed_hex?
+    Float("-0x1A") == -26.0
+  rescue ArgumentError
+    false
+  end
 end

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.9'
   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

@@ -318,7 +318,7 @@ describe LogStash::Filters::Mutate do
             convert => [ "message", "int"] #should be integer
           }
         }
-      CONFIG
+     CONFIG
 
       sample "not_really_important" do
         expect {subject}.to raise_error(LogStash::ConfigurationError, /Invalid conversion type/)
@@ -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")
@@ -1102,4 +1122,451 @@ describe LogStash::Filters::Mutate do
     end
   end
 
+  describe "convert makes type conversions" do
+    context "to string" do
+      config <<-CONFIG
+        filter {
+          mutate { convert => { "a"  => "string" } }
+        }
+      CONFIG
+
+      context "123" do
+        # Integer
+        sample({ "a" => 123 }) do
+          expect(subject.get("a")).to be_a(String).and eq("123")
+        end
+        # Float
+        sample({ "a" => Float(123.0) }) do
+          expect(subject.get("a")).to be_a(String).and eq("123.0")
+        end
+        # String
+        sample({ "a" => "123" }) do
+          expect(subject.get("a")).to be_a(String).and eq("123")
+        end
+        sample({ "a" => "0x7b" }) do
+          expect(subject.get("a")).to be_a(String).and eq("0x7b")
+        end
+        sample({ "a" => "123.0" }) do
+          expect(subject.get("a")).to be_a(String).and eq("123.0")
+        end
+        sample({ "a" => "1.230000e+02" }) do
+          expect(subject.get("a")).to be_a(String).and eq("1.230000e+02")
+        end
+      end
+
+      context "123.45" do
+        # Float
+        sample({ "a" => Float(123.45) }) do
+          expect(subject.get("a")).to be_a(String).and eq("123.45")
+        end
+        # String
+        sample({ "a" => "123.45" }) do
+          expect(subject.get("a")).to be_a(String).and eq("123.45")
+        end
+        sample({ "a" => "1.234500e+02" }) do
+          expect(subject.get("a")).to be_a(String).and eq("1.234500e+02")
+        end
+        sample({ "a" => "0x1.edcdp6" }) do
+          expect(subject.get("a")).to be_a(String).and eq( "0x1.edcdp6" )
+        end
+      end
+
+      context "16777217" do
+        # Integer
+        sample({ "a" => 16777217 }) do
+          expect(subject.get("a")).to be_a(String).and eq("16777217")
+        end
+        # Float
+        sample({ "a" => 1.6777217E7 }) do
+          expect(subject.get("a")).to be_a(String).and eq("16777217.0")
+        end
+        # String
+        sample({ "a" => "16777217" }) do
+          expect(subject.get("a")).to be_a(String).and eq("16777217")
+        end
+        sample({ "a" => "16777217.0" }) do
+          expect(subject.get("a")).to be_a(String).and eq("16777217.0")
+        end
+      end
+
+      context "2147483648" do
+        # Long
+        sample({ "a" => 2147483648 }) do
+          expect(subject.get("a")).to be_a(String).and eq("2147483648")
+        end
+        # Double
+        sample({ "a" => 2.147483648E9 }) do
+          expect(subject.get("a")).to be_a(String).and eq("2147483648.0")
+        end
+        # String
+        sample({ "a" => "2147483648" }) do
+          expect(subject.get("a")).to be_a(String).and eq("2147483648")
+        end
+        sample({ "a" => "2147483648.0" }) do
+          expect(subject.get("a")).to be_a(String).and eq("2147483648.0")
+        end
+      end
+
+      context "9007199254740993" do
+        # Long
+        sample({ "a" => 9007199254740993 }) do
+          expect(subject.get("a")).to be_a(String).and eq("9007199254740993")
+        end
+        # String
+        sample({ "a" => "9007199254740993" }) do
+          expect(subject.get("a")).to be_a(String).and eq("9007199254740993")
+        end
+        sample({ "a" => "9007199254740993.0" }) do
+          expect(subject.get("a")).to be_a(String).and eq("9007199254740993.0")
+        end
+      end
+
+
+      context "9223372036854775808" do
+        # String
+        sample({ "a" => "9223372036854775808" }) do
+          expect(subject.get("a")).to be_a(String).and eq("9223372036854775808")
+        end
+        sample({ "a" => "9223372036854775808.0" }) do
+          expect(subject.get("a")).to be_a(String).and eq("9223372036854775808.0")
+        end
+      end
+
+      context "680564693277057720000000000000000000000" do
+        # String
+        sample({ "a" => "680564693277057720000000000000000000000" }) do
+          expect(subject.get("a")).to be_a(String).and eq("680564693277057720000000000000000000000")
+        end
+        sample({ "a" => "680564693277057720000000000000000000000.0" }) do
+          expect(subject.get("a")).to be_a(String).and eq("680564693277057720000000000000000000000.0")
+        end
+      end
+    end
+
+    context "to integer" do
+      config <<-CONFIG
+        filter {
+          mutate { convert => { "a"  => "integer" } }
+        }
+      CONFIG
+
+      context "123" do
+        # Integer
+        sample({ "a" => 123 }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        # Float
+        sample({ "a" => Float(123.0) }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        # String
+        sample({ "a" => "123" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        sample({ "a" => "0x7b" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        sample({ "a" => "123.0" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        sample({ "a" => "1.230000e+02" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+      end
+
+      context "123.45" do
+        # Float
+        sample({ "a" => Float(123.45) }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        # String
+        sample({ "a" => "123.45" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        sample({ "a" => "1.234500e+02" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+        sample({ "a" => "0x1.edcdp6" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(123)
+        end
+      end
+
+      context "-123.45" do
+        # Float
+        sample({ "a" => Float(-123.45) }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(-123)
+        end
+        # String
+        sample({ "a" => "-123.45" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(-123)
+        end
+        sample({ "a" => "-1.234500e+02" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(-123)
+        end
+        sample({ "a" => "-0x1.edcdp6" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(-123)
+        end
+      end
+
+      context "16777217" do
+        # Integer
+        sample({ "a" => 16777217 }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(16777217)
+        end
+        # Double
+        sample({ "a" => 1.6777217E7 }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(16777217)
+        end
+        # String
+        sample({ "a" => "16777217" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(16777217)
+        end
+        sample({ "a" => "16777217.0" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(16777217)
+        end
+      end
+
+      context "2147483648" do
+        # Long
+        sample({ "a" => 2147483648 }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(2147483648)
+        end
+        # Double
+        sample({ "a" => 2.147483648E9 }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(2147483648)
+        end
+        # String
+        sample({ "a" => "2147483648" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(2147483648)
+        end
+        sample({ "a" => "2147483648.0" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(2147483648)
+        end
+      end
+
+      context "9007199254740993" do
+        # Long
+        sample({ "a" => 9007199254740993 }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(9007199254740993)
+        end
+        # String
+        sample({ "a" => "9007199254740993" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(9007199254740993)
+        end
+        sample({ "a" => "9007199254740993.0" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(9007199254740993)
+        end
+        sample({ "a" => "9.007199254740993e+15" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(9007199254740993)
+        end
+      end
+
+      context "9223372036854775808" do
+        # String
+        sample({ "a" => "9223372036854775808" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(9223372036854775808)
+        end
+        sample({ "a" => "9223372036854775808.0" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(9223372036854775808)
+        end
+      end
+
+      context "680564693277057720000000000000000000000" do
+        # String
+        sample({ "a" => "680564693277057720000000000000000000000" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(680564693277057720000000000000000000000)
+        end
+        sample({ "a" => "680564693277057720000000000000000000000.0" }) do
+          expect(subject.get("a")).to be_a(Integer).and eq(680564693277057720000000000000000000000)
+        end
+      end
+
+    end
+
+    context "to float" do
+      config <<-CONFIG
+        filter {
+          mutate { convert => { "a"  => "float" } }
+        }
+      CONFIG
+
+      context "123" do
+        # Integer
+        sample({ "a" => 123 }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.0)
+        end
+        # Float
+        sample({ "a" => Float(123) }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.0)
+        end
+        # String
+        sample({ "a" => "123" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.0)
+        end
+        sample({ "a" => "0x7b" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.0)
+        end
+        sample({ "a" => "123.0" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.0)
+        end
+        sample({ "a" => "1.230000e+02" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.0)
+        end
+      end
+
+      context "123.45" do
+        # Float
+        sample({ "a" => Float(123.45) }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.45)
+        end
+        # String
+        sample({ "a" => "123.45" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.45)
+        end
+        sample({ "a" => "1.234500e+02" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.45)
+        end
+        sample({ "a" => "0x1.edcdp6" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(123.4501953125)
+        end
+      end
+
+      context "-123.45" do
+        # Float
+        sample({ "a" => Float(-123.45) }) do
+          expect(subject.get("a")).to be_a(Float).and eq(-123.45)
+        end
+        # String
+        sample({ "a" => "-123.45" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(-123.45)
+        end
+        sample({ "a" => "-1.234500e+02" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(-123.45)
+        end
+        sample({ "a" => "-0x1.edcdp6" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(-123.4501953125)
+        end
+      end
+
+      context "16777217" do
+        # Integer
+        sample({ "a" => 16777217 }) do
+          expect(subject.get("a")).to be_a(Float).and eq(1.6777217E7)
+        end
+        # Float
+        sample({ "a" => 1.6777217E7 }) do
+          expect(subject.get("a")).to be_a(Float).and eq(1.6777217E7)
+        end
+        # String
+        sample({ "a" => "16777217" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(1.6777217E7)
+        end
+        sample({ "a" => "16777217.0" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(1.6777217E7)
+        end
+      end
+
+
+      context "2147483648" do
+        # Long
+        sample({ "a" => 2147483648 }) do
+          expect(subject.get("a")).to be_a(Float).and eq(2.147483648E9)
+        end
+        # Double
+        sample({ "a" => 2.147483648E9 }) do
+          expect(subject.get("a")).to be_a(Float).and eq(2.147483648E9)
+        end
+        # String
+        sample({ "a" => "2147483648" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(2.147483648E9)
+        end
+        sample({ "a" => "2147483648.0" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(2.147483648E9)
+        end
+      end
+
+      context "9007199254740993" do
+        # Long
+        sample({ "a" => 9007199254740993 }) do
+          expect(subject.get("a")).to be_a(Float).and eq(9.007199254740992E15)
+        end
+        # String
+        sample({ "a" => "9007199254740993" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(9.007199254740992E15)
+        end
+        sample({ "a" => "9007199254740993.0" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(9.007199254740992E15)
+        end
+        sample({ "a" => "9.007199254740993e+15" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(9007199254740992)
+        end
+      end
+
+      context "9223372036854775808" do
+        # String
+        sample({ "a" => "9223372036854775808" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(9.223372036854775808E18)
+        end
+        sample({ "a" => "9223372036854775808.0" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(9.223372036854775808E18)
+        end
+      end
+
+      context "680564693277057720000000000000000000000" do
+        # String
+        sample({ "a" => "680564693277057720000000000000000000000" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(6.805646932770577e+38)
+        end
+        sample({ "a" => "680564693277057720000000000000000000000.0" }) do
+          expect(subject.get("a")).to be_a(Float).and eq(6.805646932770577e+38)
+        end
+      end
+    end
+
+  end unless LogStash::Filters::Mutate.is_lenient_version? # only test type conversions in v8.14+
+
+  describe "parse_signed_hex_str" do
+    subject { LogStash::Filters::Mutate.new({ }) }
+
+    context 'hexadecimal integers' do
+      it 'parses positive hex integer' do
+        expect(subject.send(:parse_signed_hex_str, '0x1A')).to eq(26.0)
+      end
+
+      it 'parses negative hex integer' do
+        expect(subject.send(:parse_signed_hex_str, '-0x1A')).to eq(-26.0)
+        expect(subject.send(:parse_signed_hex_str,'-0xFF')).to eq(-255.0)
+      end
+
+      it 'ignores case in hex prefix' do
+        expect(subject.send(:parse_signed_hex_str,'0x1A')).to eq(26.0)
+        expect(subject.send(:parse_signed_hex_str,'-0x1A')).to eq(-26.0)
+      end
+    end
+
+    context 'hexadecimal floats' do
+      it 'parses positive hex float' do
+        expect(subject.send(:parse_signed_hex_str,'0x1.8p+1')).to eq(3.0)
+      end
+
+      it 'parses negative hex float' do
+        expect(subject.send(:parse_signed_hex_str,'-0x1.8p+1')).to eq(-3.0)
+        expect(subject.send(:parse_signed_hex_str,'-0x1.2p+2')).to eq(-4.5)
+      end
+    end
+
+    context 'non-hex strings' do
+      it 'returns nil for decimal numbers' do
+        expect(subject.send(:parse_signed_hex_str,'123')).to be_nil
+        expect(subject.send(:parse_signed_hex_str,'-456')).to be_nil
+        expect(subject.send(:parse_signed_hex_str,'1.23')).to be_nil
+      end
+
+      it 'returns nil for random strings' do
+        expect(subject.send(:parse_signed_hex_str,'abc')).to be_nil
+        expect(subject.send(:parse_signed_hex_str,'0b1010')).to be_nil
+      end
+    end
+  end unless LogStash::Filters::Mutate.is_lenient_version?
+
 end

metadata

@@ -1,16 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-mutate
 version: !ruby/object:Gem::Version
-  version: 3.5.7
+  version: 3.5.9
 platform: ruby
 authors:
 - Elastic
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-11 00:00:00.000000000 Z
+date: 2025-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
+  name: logstash-core-plugin-api
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -19,9 +19,8 @@ dependencies:
     - - "<="
       - !ruby/object:Gem::Version
         version: '2.99'
-  name: logstash-core-plugin-api
-  prerelease: false
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -31,56 +30,56 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.99'
 - !ruby/object:Gem::Dependency
+  name: logstash-patterns-core
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: logstash-patterns-core
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: logstash-filter-grok
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: logstash-filter-grok
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: logstash-codec-plain
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: logstash-codec-plain
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: logstash-devutils
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: logstash-devutils
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -111,7 +110,6 @@ licenses:
 metadata:
   logstash_plugin: 'true'
   logstash_group: filter
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -126,8 +124,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.33
-signing_key:
+rubygems_version: 3.6.3
 specification_version: 4
 summary: Performs mutations on fields
 test_files: