refactor: merge rails and ruby logger detector (#180)

Bearer · Dec 1, 2022 · 7903ce5 · 7903ce5
1 parent cc94a7b
commit 7903ce5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 21 deletions.
diff --git a/integration/flags/.snapshots/TestInitCommand-init b/integration/flags/.snapshots/TestInitCommand-init
@@ -38,20 +38,6 @@ scan:
             root_lowercase: false
             metavars: {}
             stored: false
-        detect_rails_logger:
-            disabled: false
-            type: risk
-            languages:
-                - ruby
-            patterns:
-                - |
-                  Rails.logger.info(<$ARGUMENT>)
-            param_parenting: false
-            processors: []
-            root_singularize: false
-            root_lowercase: false
-            metavars: {}
-            stored: false
         detect_rails_session:
             disabled: false
             type: risk
@@ -74,6 +60,8 @@ scan:
             patterns:
                 - |
                   logger.info(<$ARGUMENT>)
+                - |
+                  Rails.logger.info(<$ARGUMENT>)
             param_parenting: false
             processors: []
             root_singularize: false

diff --git a/pkg/commands/process/settings/custom_detector.yml b/pkg/commands/process/settings/custom_detector.yml
@@ -32,18 +32,13 @@ ruby_file_detection:
   param_parenting: true
   metavars: {}
   stored: false
-detect_rails_logger:
-  type: "risk"
-  patterns:
-    - |
-      Rails.logger.info(<$ARGUMENT>)
-  languages:
-    - ruby
 detect_ruby_logger:
   type: "risk"
   patterns:
     - |
       logger.info(<$ARGUMENT>)
+    - |
+      Rails.logger.info(<$ARGUMENT>)
   languages:
     - ruby
 detect_rails_session: