Use regexp type

Signed-off-by: Kenji Okimoto <[email protected]>
fluent · Apr 5, 2018 · 72cff25 · 72cff25
1 parent de16030
commit 72cff25
Showing 1 changed file with 4 additions and 28 deletions.
diff --git a/lib/fluent/plugin/filter_grep.rb b/lib/fluent/plugin/filter_grep.rb
@@ -60,25 +60,13 @@ def initialize
         desc "The field name to which the regular expression is applied."
         config_param :key, :string
         desc "The regular expression."
-        config_param :pattern do |value|
-          if value.start_with?("/") and value.end_with?("/")
-            Regexp.compile(value[1..-2])
-          else
-            Regexp.compile(value)
-          end
-        end
+        config_param :pattern, :regexp
       end
       config_section :exclude, param_name: :excludes, multi: true do
         desc "The field name to which the regular expression is applied."
         config_param :key, :string
         desc "The regular expression."
-        config_param :pattern do |value|
-          if value.start_with?("/") and value.end_with?("/")
-            Regexp.compile(value[1..-2])
-          else
-            Regexp.compile(value)
-          end
-        end
+        config_param :pattern, :regexp
       end
     end
 
@@ -87,25 +75,13 @@ def initialize
         desc "The field name to which the regular expression is applied."
         config_param :key, :string
         desc "The regular expression."
-        config_param :pattern do |value|
-          if value.start_with?("/") and value.end_with?("/")
-            Regexp.compile(value[1..-2])
-          else
-            Regexp.compile(value)
-          end
-        end
+        config_param :pattern, :regexp
       end
       config_section :exclude, param_name: :excludes, multi: true do
         desc "The field name to which the regular expression is applied."
         config_param :key, :string
         desc "The regular expression."
-        config_param :pattern do |value|
-          if value.start_with?("/") and value.end_with?("/")
-            Regexp.compile(value[1..-2])
-          else
-            Regexp.compile(value)
-          end
-        end
+        config_param :pattern, :regexp
       end
     end