Fix for rotate_age where Fluentd passes as Symbol (#4311)

Fix for rotate_age where Fluentd passes as Symbol while Ruby Logger expects String --------- Signed-off-by: mrudrego <[email protected]> Signed-off-by: mrudrego <[email protected]>
fluent · Oct 18, 2023 · dd1a6e5 · dd1a6e5
1 parent fcbcb56
commit dd1a6e5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/fluent/system_config.rb b/lib/fluent/system_config.rb
@@ -62,7 +62,7 @@ class SystemConfig
       config_param :time_format, :string, default: '%Y-%m-%d %H:%M:%S %z'
       config_param :rotate_age, default: nil do |v|
         if Fluent::Log::LOG_ROTATE_AGE.include?(v)
-          v.to_sym
+          v
         else
           begin
             Integer(v)

diff --git a/test/config/test_system_config.rb b/test/config/test_system_config.rb
@@ -151,7 +151,7 @@ def parse_text(text)
       data('daily' => "daily",
            'weekly' => 'weekly',
            'monthly' => 'monthly')
-      test "symbols for rotate_age" do |age|
+      test "strings for rotate_age" do |age|
         conf = parse_text(<<-EOS)
           <system>
             <log>
@@ -160,7 +160,7 @@ def parse_text(text)
           </system>
         EOS
         sc = Fluent::SystemConfig.new(conf)
-        assert_equal(age.to_sym, sc.log.rotate_age)
+        assert_equal(age, sc.log.rotate_age)
       end
 
       test "numeric number for rotate age" do