Fix stack trace causing request blocking

DataDog · Jan 17, 2025 · 8d05d95 · 8d05d95
1 parent e50bcb3
commit 8d05d95
Show file tree

Hide file tree

Showing 7 changed files with 7 additions and 7 deletions.
diff --git a/lib/datadog/appsec/contrib/graphql/reactive/multiplex.rb b/lib/datadog/appsec/contrib/graphql/reactive/multiplex.rb
@@ -35,7 +35,7 @@ def self.subscribe(engine, context)
                 next if result.status != :match
 
                 yield result
-                throw(:block, true) unless result.actions.empty?
+                throw(:block, true) if result.actions.include?('block')
               end
             end
           end

diff --git a/lib/datadog/appsec/contrib/rack/reactive/request.rb b/lib/datadog/appsec/contrib/rack/reactive/request.rb
@@ -58,7 +58,7 @@ def self.subscribe(engine, context)
                 next if result.status != :match
 
                 yield result
-                throw(:block, true) unless result.actions.empty?
+                throw(:block, true) if result.actions.include?('block')
               end
             end
           end

diff --git a/lib/datadog/appsec/contrib/rack/reactive/request_body.rb b/lib/datadog/appsec/contrib/rack/reactive/request_body.rb
@@ -36,7 +36,7 @@ def self.subscribe(engine, context)
                 next if result.status != :match
 
                 yield result
-                throw(:block, true) unless result.actions.empty?
+                throw(:block, true) if result.actions.include?('block')
               end
             end
           end

diff --git a/lib/datadog/appsec/contrib/rack/reactive/response.rb b/lib/datadog/appsec/contrib/rack/reactive/response.rb
@@ -42,7 +42,7 @@ def self.subscribe(engine, context)
                 next if result.status != :match
 
                 yield result
-                throw(:block, true) unless result.actions.empty?
+                throw(:block, true) if result.actions.include?('block')
               end
             end
           end

diff --git a/lib/datadog/appsec/contrib/rails/reactive/action.rb b/lib/datadog/appsec/contrib/rails/reactive/action.rb
@@ -42,7 +42,7 @@ def self.subscribe(engine, context)
                 next if result.status != :match
 
                 yield result
-                throw(:block, true) unless result.actions.empty?
+                throw(:block, true) if result.actions.include?('block')
               end
             end
           end

diff --git a/lib/datadog/appsec/contrib/sinatra/reactive/routed.rb b/lib/datadog/appsec/contrib/sinatra/reactive/routed.rb
@@ -37,7 +37,7 @@ def self.subscribe(engine, context)
                 next if result.status != :match
 
                 yield result
-                throw(:block, true) unless result.actions.empty?
+                throw(:block, true) if result.actions.include?('block')
               end
             end
           end

diff --git a/lib/datadog/appsec/monitor/reactive/set_user.rb b/lib/datadog/appsec/monitor/reactive/set_user.rb
@@ -35,7 +35,7 @@ def self.subscribe(engine, context)
               next if result.status != :match
 
               yield result
-              throw(:block, true) unless result.actions.empty?
+              throw(:block, true) if result.actions.include?('block')
             end
           end
         end