rails-guarddog 0.1.11 → 0.1.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 859dc0da3cfd03afa3ceea5be6d2feba0f3f49786f109fb82c0a5b1c379c0837
-  data.tar.gz: a0251fa578fb369a59316ff36e62e4409cbfb0b08bdc4c4be1d2d75b764c1411
+  metadata.gz: b5176cbe7acb78424e6f30a57cb8c203d5de27d9c2e12627034ee33a723c3095
+  data.tar.gz: 22408c90ed568ef535fced6db89e92a96c7b79aec37cc81631abb7a0a5393498
 SHA512:
-  metadata.gz: 3394a5d46d8ed065003722151b6f5d74361fac8e9656a3c39760e02fc25e581c474cb3246aa9bb8337530a50f70d492dfc77127b1770f7fa82bc6d01460073e5
-  data.tar.gz: f52bea6bb7b20187c2ab4a1751267f5987116f1839657bda880dfb93e18a96a8840e4621fde4b490c4d4d318c9f42fbe453be04c6514aa0a42c26b4293a40840
+  metadata.gz: fc28d9a40e236b423c809aaddefed91c6de16a6108563e1d47512f8d2d5ef91a0caac2e3113c240e6d55483b272c87603a8dfa190bcdb14213604166bc547eb4
+  data.tar.gz: 1d0e968eaf094a70a7854dfe5223c048a02f445f475345b3d5edb0e7a12448c398c960d18b1a5f0d6fa9a9dbbd2f8a1813141739a74cf9a55fcb4281a1fea719

data/README.md

@@ -104,14 +104,17 @@ That's it! Scan your entire Rails app for security vulnerabilities.
 - **DoS/ReDoS** — Unbounded queries, dangerous regex patterns ⭐ ENHANCED in v0.1.8
 - **Supply Chain** — Typosquatted gems using Levenshtein distance ⭐ IMPROVED in v0.1.8
 
-> **False-positive protection (v0.1.10)**
-> - Patterns use word boundaries — `no_token:` or `reset_password_instructions:` in locale
+> **False-positive protection (v0.1.10 - v0.1.13)**
+> - **Secrets**: Patterns use word boundaries — `no_token:` or `reset_password_instructions:` in locale
 >   files will **not** trigger the secrets checker.
-> - Secret values must be space-free (`[^\s'"]{6,}`) — human-readable sentences like
+> - **Secrets**: Secret values must be space-free (`[^\s'"]{6,}`) — human-readable sentences like
 >   `"You can't access this page..."` are never flagged.
-> - `config/locales/` files are skipped entirely — i18n translations are not credentials.
-> - `spec/` and `test/` trees suppress the secrets checker by default (configurable via
+> - **Secrets**: `config/locales/` files are skipped entirely — i18n translations are not credentials.
+> - **Secrets**: `spec/` and `test/` trees suppress the secrets checker by default (configurable via
 >   `ignored_checks`).
+> - **DoS**: Smart multiline `.limit()` check detection (v0.1.12) — lookahead up to 3 lines handles chained
+>   `.limit()` calls on subsequent lines or reassignment to the same variable, e.g. `things = Thing.all; things = things.limit(100)` or `.limit(...)` on the next line.
+> - **Ignore File**: Support for `.guarddogignore` (v0.1.13) — allows excluding complete files/directories or ignoring specific checkers for specific files/directories.
 
 
 ## 📊 Example Output
@@ -119,7 +122,7 @@ That's it! Scan your entire Rails app for security vulnerabilities.
 ### Console Report
 ```
 ============================================================
-          Rails GuardDog Security Report v0.1.10
+          Rails GuardDog Security Report v0.1.13
 ============================================================
 
 [CRITICAL] (5 findings)
@@ -223,6 +226,22 @@ GuardDog already silences `secrets` in `spec/` and `test/` by default. To add
 more suppressions or override the defaults, set `ignored_checks` in your
 initializer (see example above).
 
+### Ignoring findings with `.guarddogignore`
+
+You can add a `.guarddogignore` file in the root of your Rails project to exclude entire files/directories or ignore specific checkers for specific files.
+
+- To exclude a file or directory completely (silence all checkers), list the path:
+  ```
+  db/seeds.rb
+  vendor/
+  ```
+
+- To skip only specific checkers for certain files or directories, separate the path and the comma-separated list of checkers with a `:`:
+  ```
+  db/seeds.rb: secrets
+  app/controllers/admin/*: sql_injection, csrf
+  ```
+
 ---
 
 
@@ -290,6 +309,6 @@ MIT License - Free to use and modify.
 
 ---
 
-**Rails GuardDog v0.1.10 — Production Ready**
+**Rails GuardDog v0.1.13 — Production Ready**
 
 *Beyond brakeman. Detect what others miss.* 🐕🔒

data/lib/rails/guarddog/checkers/base_checker.rb

@@ -43,9 +43,10 @@ module Rails
 
           ignored = @configuration.ignored_checks || {}
           name    = checker_identifier
+          rel_file = relative_path(file)
 
           ignored.any? do |path_pattern, checker_names|
-            file.include?(path_pattern.to_s) &&
+            rel_file.include?(path_pattern.to_s) &&
               checker_names.map(&:to_s).include?(name)
           end
         end
@@ -63,9 +64,24 @@ module Rails
         def glob_files(pattern)
           excluded = @configuration&.excluded_paths || []
           Dir.glob(File.join(@root, pattern)).reject do |file|
-            excluded.any? { |path| file.include?(path) }
+            rel_file = relative_path(file)
+            excluded.any? { |path| rel_file.include?(path) }
           end
         end
+
+        private
+
+        def relative_path(file)
+          file_normalized = file.to_s.gsub('\\', '/')
+          root_normalized = @root.to_s.gsub('\\', '/')
+          
+          if file_normalized.start_with?(root_normalized)
+            file_normalized.sub(/\A#{Regexp.escape(root_normalized)}\/?/, '')
+          else
+            file_normalized
+          end
+        end
+
       end
     end
   end

data/lib/rails/guarddog/checkers/dos_checker.rb

@@ -4,20 +4,19 @@ module Rails
       class DosChecker < BaseChecker
         # Matches .all only when it is the final call on an ActiveRecord-style
         # model class (PascalCase constant), e.g.:
-        #   User.all           => flagged
+        #   User.all            => flagged
         #   Post.where(...).all => flagged
         # But NOT:
-        #   current_user.permissions.track_manage.all  => NOT flagged (chained on object)
-        #   collection.all? { }                        => NOT flagged (.all? method)
-        #   track_manage.all                           => NOT flagged (lowercase receiver)
+        #   current_user.permissions.track_manage.all => NOT flagged (lowercase receiver)
+        #   collection.all? { }                       => NOT flagged (.all? method)
         UNBOUNDED_QUERY_PATTERN = /
           (?:
-            [A-Z][A-Za-z0-9_]*   # PascalCase model class  e.g. User, TrackItem
-            (?:\.[a-z_]+\(.*?\))* # optional chained scopes e.g. .where(...).order(...)
+            [A-Z][A-Za-z0-9_]*    # PascalCase model class  e.g. User, TrackItem
+            (?:\.[a-z_]+\(.*?\))*  # optional chained scopes e.g. .where(...).order(...)
           )
-          \.all                   # the .all call
-          (?!\?)                  # not .all? (Enumerable method, not AR query)
-          \s*(?:[,)\]#\n]|$)     # followed by end-of-expression
+          \.all                    # the .all call
+          (?!\?)                   # not .all? (Enumerable method, not AR query)
+          \s*(?:[,)\]#\n]|$)      # followed by end-of-expression (not .limit etc.)
         /x.freeze
 
         # Patterns known to be dangerous nested regex (ReDoS)
@@ -26,13 +25,18 @@ module Rails
           /match\?.*\(.+\*\+/
         ].freeze
 
+        # How many lines ahead to search for a .limit() call.
+        # Covers both chained-on-next-line and separate-assignment styles.
+        LOOKAHEAD_LINES = 3
+
         def run
           glob_files('app/**/*.rb').each do |file|
-            content = File.read(file) rescue next
-            content.each_line.with_index do |line, idx|
+            lines = File.readlines(file) rescue next
+
+            lines.each_with_index do |line, idx|
               next if line.strip.start_with?('#')
 
-              if line.match?(UNBOUNDED_QUERY_PATTERN)
+              if line.match?(UNBOUNDED_QUERY_PATTERN) && !limit_applied?(lines, idx, line)
                 add_finding(
                   severity: :high,
                   message: "Potential DoS: unbounded database query without limit",
@@ -57,6 +61,46 @@ module Rails
           end
           findings
         end
+
+        private
+
+        # Returns true if a .limit() call is found either:
+        #   1. On the same line as .all      (e.g. Thing.all.limit(100))
+        #   2. Chained on the very next line  (e.g. Thing.all\n  .limit(100))
+        #   3. Applied to the assigned variable within LOOKAHEAD_LINES lines
+        #      (e.g. things = Thing.all \n things = things.limit(100))
+        def limit_applied?(lines, current_idx, current_line)
+          # 1. Same-line limit (Thing.all.limit(100)) — already excluded by
+          #    UNBOUNDED_QUERY_PATTERN's end-of-expression anchor, but double-check.
+          return true if current_line.match?(/\.limit\s*\(/)
+
+          # Extract the variable being assigned, supporting local vars, instance
+          # vars (@foo) and class-instance vars (@@foo).
+          assigned_var = current_line.match(/([@]{0,2}\w+)\s*=\s*[A-Z]/)&.[](1)
+
+          lookahead = lines[(current_idx + 1), LOOKAHEAD_LINES] || []
+
+          lookahead.each do |next_line|
+            stripped = next_line.lstrip
+
+            # 2. Chained on next line: the continuation starts with .limit
+            return true if stripped.start_with?('.limit')
+
+            # 3. Assigned variable subsequently limited:
+            #      things   = things.limit(100)   => local var (word boundary ok)
+            #      @things  = @things.limit(100)  => instance var (@ is non-word, use look-behind)
+            #      things.limit(100)
+            if assigned_var
+              # Build an anchor that works for both `@var` and plain `var`:
+              # require that the var is preceded by start-of-line, whitespace or `=`.
+              anchor  = '(?:^|[\s=(])'
+              escaped = Regexp.escape(assigned_var)
+              return true if next_line.match?(/#{anchor}#{escaped}(?:\s*=\s*#{escaped})?\.limit\s*\(/)
+            end
+          end
+
+          false
+        end
       end
     end
   end

data/lib/rails/guarddog/scanner.rb

@@ -4,11 +4,12 @@ module Rails
       attr_accessor :configuration, :findings
 
       def initialize(config = nil)
-        @configuration = config || Configuration.new
+        @configuration = config || default_configuration
         @findings = []
       end
 
       def run
+        load_ignore_file
         load_checkers.each do |checker_class|
           checker = checker_class.new(@configuration)
           checker.run
@@ -19,6 +20,41 @@ module Rails
 
       private
 
+      def default_configuration
+        if defined?(Rails) && Rails.application && Rails.application.config.respond_to?(:guarddog)
+          Rails.application.config.guarddog
+        else
+          Configuration.new
+        end
+      end
+
+      def load_ignore_file
+        ignore_file_path = File.join(@configuration.root, '.guarddogignore')
+        return unless File.exist?(ignore_file_path)
+
+        @configuration.ignored_checks ||= {}
+        @configuration.excluded_paths ||= []
+
+        File.foreach(ignore_file_path).each_with_index do |line, idx|
+          line = line.strip
+          next if line.empty? || line.start_with?('#')
+
+          if line.include?(':')
+            path_pattern, checkers_str = line.split(':', 2)
+            path_pattern = path_pattern.strip.gsub('\\', '/')
+            checkers = checkers_str.split(',').map { |c| c.strip.downcase }
+
+            @configuration.ignored_checks[path_pattern] ||= []
+            @configuration.ignored_checks[path_pattern].concat(checkers)
+            @configuration.ignored_checks[path_pattern].uniq!
+          else
+            path_pattern = line.gsub('\\', '/')
+            @configuration.excluded_paths << path_pattern
+            @configuration.excluded_paths.uniq!
+          end
+        end
+      end
+
       def load_checkers
         checker_map = {
           'sql_injection'  => Checkers::SqlInjectionChecker,

data/lib/rails/guarddog/version.rb

@@ -1,5 +1,5 @@
 module Rails
   module Guarddog
-    VERSION = "0.1.11"
+    VERSION = "0.1.13"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails-guarddog
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.13
 platform: ruby
 authors:
 - Security Team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-07-13 00:00:00.000000000 Z
+date: 2026-08-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties