sqreen 1.23.2 → 1.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e1881ab1def0e4a82ccfe791c150b0213517084619d71f5c942a0d14a0189e1
-  data.tar.gz: beeef4f164a84efbc00824a5c63926da6a4e3fcc5952577442061e1c509e8fda
+  metadata.gz: b43caee6a7702bfde6a56f9a3d9ad735de10b5a214ce5d097c3c26c3527f5918
+  data.tar.gz: 803451b19b5277384c9e74b186e00e0a1d0d8a8a77cb5483cf486088a1705b0d
 SHA512:
-  metadata.gz: b1d1ea43737baa0db830e556fc835f93104dc70465fc5695372c5a922e8108cb9e657e915cc7ffe06f655dc632b0d5e56daf659e0c6ef499b66484fcd6d33076
-  data.tar.gz: 98b0e405e39de271a9d8988692ea037b5f462b2af9314d384b7ca7dfec9dce43229b1aa317fbb8fccde84f1cd743d90637dabde392a2c7bfe88d6a6353e25235
+  metadata.gz: 2d35106412544c74a99bcf5eb39b1bc33d95720efbce196e139c15f40c1259b1aaf4646cab127849b9235d3f1cf0ce421c01859eb1c4bc6076683d3f7beeef33
+  data.tar.gz: b0f5452dda0d2702397968e103313bf910bb996c09064354be78fcabb6492dbb15d898aa1caf5835307a5f21b878550fae59826fbc347382772cba1823cefea2

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.24.0
+
+* Add Sqreen event correlation with Datadog traces
+
 ## 1.23.2
 
 * Fix compatibility with NewRelic for attack events

data/lib/sqreen/events/attack.rb

@@ -63,6 +63,14 @@ module Sqreen
       payload['context']['backtrace']
     end
 
+    def datadog_trace_id
+      payload['context']['datadog_trace_id']
+    end
+
+    def datadog_span_id
+      payload['context']['datadog_span_id']
+    end
+
     def enqueue
       Sqreen.queue.push(self)
     end

data/lib/sqreen/frameworks/generic.rb

@@ -173,7 +173,18 @@ module Sqreen
           :remote_port => req.env['REMOTE_PORT'],
           :remote_ip => remote_addr,
           :client_ip => client_ip,
-        }
+        }.tap do |h|
+          h.merge!(
+            :datadog_trace_id => datadog_span.trace_id,
+            :datadog_span_id => datadog_span.span_id,
+          ) if datadog_span
+        end
+      end
+
+      def datadog_span
+        return unless defined?(Datadog) && (tracer = Datadog.tracer)
+
+        tracer.active_span
       end
 
       def response_infos

data/lib/sqreen/signals/conversions.rb

@@ -44,11 +44,17 @@ module Sqreen
         # XXX: not used because we don't use Sqreen::Attack
         def convert_attack(attack)
           # no need to set actor/context as we only include them in request records/traces
+          location_h = {}
+          location_h.merge!(stack_trace: attack.backtrace) if attack.backtrace
+          location_h.merge!(datadog_trace_id: datadog_trace_id) if attack.datadog_trace_id
+          location_h.merge!(datadog_span_id: datadog_span_id) if attack.datadog_span_id
+          location = Kit::Signals::Location.new(location_h) unless location_h.empty?
+
           Kit::Signals::Specialized::Attack.new(
             signal_name: "sq.agent.attack.#{attack.attack_type}",
             source: "sqreen:rule:#{attack.rulespack_id}:#{attack.rule_name}",
             time: attack.time,
-            location: Kit::Signals::Location.new(stack_trace: attack.backtrace),
+            location: location,
             payload: Kit::Signals::Specialized::Attack::Payload.new(
               test: attack.test?,
               block: attack.block?,
@@ -59,11 +65,17 @@ module Sqreen
 
         # see Sqreen::Rules::RuleCB.record_event
         def convert_unstructured_attack(payload)
+          location_h = {}
+          location_h.merge!(stack_trace: payload[:backtrace]) if payload[:backtrace]
+          location_h.merge!(datadog_trace_id: payload[:datadog_trace_id]) if payload[:datadog_span_id]
+          location_h.merge!(datadog_span_id: payload[:datadog_span_id]) if payload[:datadog_span_id]
+          location = Kit::Signals::Location.new(location_h) unless location_h.empty?
+
           Kit::Signals::Specialized::Attack.new(
             signal_name: "sq.agent.attack.#{payload[:attack_type]}",
             source: "sqreen:rule:#{payload[:rulespack_id]}:#{payload[:rule_name]}",
             time: payload[:time],
-            location: (Kit::Signals::Location.new(stack_trace: payload[:backtrace]) if payload[:backtrace]),
+            location: location,
             payload: Kit::Signals::Specialized::Attack::Payload.new(
               test: payload[:test],
               block: payload[:block],
@@ -234,6 +246,9 @@ module Sqreen
               status: resp_payload[:status],
               content_length: resp_payload[:content_length],
               content_type: resp_payload[:content_type],
+              # datadog
+              datadog_trace_id: req_payload[:datadog_trace_id],
+              datadog_span_id: req_payload[:datadog_span_id],
             }
           )
         end

data/lib/sqreen/version.rb

@@ -4,5 +4,5 @@
 # Please refer to our terms for more information: https://www.sqreen.com/terms.html
 
 module Sqreen
-  VERSION = '1.23.2'.freeze
+  VERSION = '1.24.0'.freeze
 end

data/lib/sqreen/weave/legacy/instrumentation.rb

@@ -244,6 +244,10 @@ class Sqreen::Weave::Legacy::Instrumentation
 
         # shrinkwrap_timer = Sqreen::Graft::Timer.new('weave,shrinkwrap')
         # shrinkwrap_timer.start
+        if defined?(Datadog)
+          datadog_span = Datadog.tracer.active_root_span
+          Sqreen::Weave.logger.debug { "request datadog:true span_id:#{datadog_span.span_id} parent_id:#{datadog_span.parent_id} trace_id:#{datadog_span.trace_id}" }
+        end
 
         request_timer = Sqreen::Graft::Timer.new("request")
         request_timer.start
@@ -269,6 +273,7 @@ class Sqreen::Weave::Legacy::Instrumentation
           timed_level: timed_level,
           skipped_callbacks: [],
           # timed_shrinkwrap: shrinkwrap_timer,
+          datadog_span: datadog_span,
         }
 
         # shrinkwrap_timer.stop

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sqreen
 version: !ruby/object:Gem::Version
-  version: 1.23.2
+  version: 1.24.0
 platform: ruby
 authors:
 - Sqreen
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-29 00:00:00.000000000 Z
+date: 2021-04-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sqreen-backport
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.2
+        version: 0.2.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.2
+        version: 0.2.3
 - !ruby/object:Gem::Dependency
   name: sq_mini_racer
   requirement: !ruby/object:Gem::Requirement
@@ -47,7 +47,7 @@ dependencies:
         version: '0.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.4.a
+        version: 0.5.a
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -57,7 +57,7 @@ dependencies:
         version: '0.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.4.a
+        version: 0.5.a
 - !ruby/object:Gem::Dependency
   name: libsqreen
   requirement: !ruby/object:Gem::Requirement