lapsoss 0.4.5 → 0.4.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5db26196bdbf6e69d90c126041f7310681e240ef22e61f0a3aee12035f7616da
-  data.tar.gz: 93a2c75b6af04e776aca9c24f3704ddba51b4a2432d5d6db0f736874bef17c69
+  metadata.gz: bc2f5c5b2a06b525bb921e683c87a201e93507c795b6cff7e4cddc33d3716880
+  data.tar.gz: 3d5bce60161f233758b0f39fdf54197ac7c5b0db921709ac3e1a4e54531bf669
 SHA512:
-  metadata.gz: 38d9d2d1c1e2f67997b71cae8776b2c7ef8b3aca5d13d563080304c63b84b1db227eb923618b1ae078223cfc20dfebf5d1620dcf01eed92ee62d53461bf36e1e
-  data.tar.gz: 1831cda3198ad594c850fd5579fed77aa9097ee4f497dd81374bb265ee237addc4d31fab0447d6164cff6fb64695869eba42366e0820647725d0725566158ff8
+  metadata.gz: 569ae7372bbe9f3984c0cb1b13074e1442af9f34cb21e74124f04db1a62227a7ac08c8d856d08c739e9da05a799c288260b72e62e8e7df44f5fa853bfacf5d7b
+  data.tar.gz: 6e17e4ffdd5d757fb70414da87e188ee12dbc1469bb37ac978554c2fc5146e5f86ebb257daefe37748e3129140d402d34c517ac1c756aa56c56defdd41a159e9

data/lib/lapsoss/adapters/concerns/http_delivery.rb

@@ -122,7 +122,7 @@ module Lapsoss
         end
 
         def user_agent
-          "Lapsoss/#{Lapsoss::VERSION} Ruby/#{RUBY_VERSION} Rails/#{Rails.version if defined?(Rails)}"
+          Base::USER_AGENT
         end
       end
     end

data/lib/lapsoss/adapters/sentry_adapter.rb

@@ -2,6 +2,7 @@
 
 require "active_support/core_ext/object/blank"
 require "uri"
+require "lapsoss/runtime_context"
 
 module Lapsoss
   module Adapters
@@ -80,7 +81,7 @@ module Lapsoss
           content_type: "application/json"
         }
 
-        item_payload = build_envelope_wrapper(event)
+        item_payload = build_sentry_event(event)
 
         # Sentry envelope is newline-delimited JSON
         [
@@ -90,6 +91,88 @@ module Lapsoss
         ].join("\n")
       end
 
+      # Build Sentry-compliant event structure
+      def build_sentry_event(event)
+        context = RuntimeContext.current
+        event_id = event.fingerprint.presence || SecureRandom.uuid
+
+        base_event = {
+          event_id: event_id,
+          timestamp: format_timestamp(event.timestamp),
+          platform: "ruby",
+          level: map_level(event.level),
+          environment: event.environment.presence || "production",
+          release: context.release,
+          server_name: context.server_name,
+          modules: context.modules,
+          contexts: context.to_contexts,
+          tags: event.tags.presence,
+          user: event.user_context.presence,
+          extra: event.extra.presence,
+          breadcrumbs: format_breadcrumbs(event.breadcrumbs),
+          sdk: {
+            name: "sentry.ruby",
+            version: Lapsoss::VERSION
+          }
+        }.compact_blank
+
+        # Add event-specific data
+        case event.type
+        when :exception
+          base_event.merge(build_exception_envelope(event))
+        when :message
+          base_event.merge(
+            message: event.message,
+            level: map_level(event.level)
+          )
+        else
+          base_event
+        end
+      end
+
+      def build_exception_envelope(event)
+        {
+          exception: {
+            values: [ {
+              type: event.exception_type,
+              value: event.exception_message,
+              module: nil,
+              thread_id: Thread.current.object_id,
+              stacktrace: build_sentry_stacktrace(event),
+              mechanism: { type: "generic", handled: true }
+            } ]
+          },
+          threads: {
+            values: [ {
+              id: Thread.current.object_id,
+              name: Thread.current.name,
+              crashed: true,
+              current: true
+            } ]
+          }
+        }
+      end
+
+      def build_sentry_stacktrace(event)
+        return nil unless event.has_backtrace?
+
+        frames = event.backtrace_frames.map do |frame|
+          {
+            filename: frame.filename,
+            abs_path: frame.absolute_path || frame.filename,
+            function: frame.method_name || frame.function,
+            lineno: frame.line_number,
+            in_app: frame.in_app,
+            pre_context: frame.code_context&.dig(:pre_context),
+            context_line: frame.code_context&.dig(:context_line),
+            post_context: frame.code_context&.dig(:post_context)
+          }.compact
+        end
+
+        # Sentry expects frames in reverse order (oldest to newest)
+        { frames: frames.reverse }
+      end
+
       # Override serialization for Sentry's envelope format
       def serialize_payload(envelope_string)
         # Sentry envelopes are already formatted, just compress if needed

data/lib/lapsoss/backtrace_frame.rb

@@ -3,6 +3,7 @@
 module Lapsoss
   BacktraceFrame = Data.define(
     :filename,
+    :absolute_path,
     :line_number,
     :method_name,
     :in_app,
@@ -19,6 +20,7 @@ module Lapsoss
     def to_h
       {
         filename: filename,
+        absolute_path: absolute_path,
         line_number: line_number,
         method: method_name,
         function: function,
@@ -30,9 +32,13 @@ module Lapsoss
     end
 
     def add_code_context(processor, context_lines = 3)
-      return unless filename && line_number && File.exist?(filename)
+      return unless line_number
 
-      with(code_context: processor.get_code_context(filename, line_number, context_lines))
+      # Use absolute path if available, otherwise try filename
+      path_to_read = absolute_path || filename
+      return unless path_to_read
+
+      with(code_context: processor.get_code_context(path_to_read, line_number, context_lines))
     end
 
     def valid?

data/lib/lapsoss/backtrace_frame_factory.rb

@@ -75,10 +75,14 @@ module Lapsoss
       filename, line_number, method_name, function, module_name, block_info = parse_line_components
 
       in_app = determine_app_status(filename)
-      filename = normalize_path(filename) if filename
+
+      # Keep both absolute and normalized paths
+      absolute_path = filename
+      normalized_filename = normalize_path(filename) if filename
 
       BacktraceFrame.new(
-        filename: filename,
+        filename: normalized_filename,
+        absolute_path: absolute_path,
         line_number: line_number,
         method_name: method_name,
         in_app: in_app,

data/lib/lapsoss/backtrace_processor.rb

@@ -196,8 +196,7 @@ module Lapsoss
 
     # Get code context around a specific line number using ActiveSupport::Cache
     def get_code_context(filename, line_number, context_lines = 3)
-      return nil unless filename && File.exist?(filename)
-      return nil if File.size(filename) > (@config[:max_file_size] || (1024 * 1024))
+      return nil unless filename
 
       lines = @file_cache.fetch(filename) do
         read_file_safely(filename)
@@ -229,6 +228,9 @@ module Lapsoss
     private
 
     def read_file_safely(filename)
+      # Don't read huge files
+      return [] if File.exist?(filename) && File.size(filename) > (@config[:max_file_size] || (1024 * 1024))
+
       File.readlines(filename, chomp: true)
     rescue StandardError
       []

data/lib/lapsoss/runtime_context.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+require "rbconfig"
+require "socket"
+
+module Lapsoss
+  # Boot-time context collection using Data class
+  RuntimeContext = Data.define(:os, :runtime, :modules, :server_name, :release) do
+    def self.current
+      @current ||= new(
+        os: collect_os_context,
+        runtime: collect_runtime_context,
+        modules: collect_modules,
+        server_name: collect_server_name,
+        release: collect_release
+      )
+    end
+
+    def self.collect_os_context
+      {
+        name: RbConfig::CONFIG["host_os"],
+        version: `uname -r 2>/dev/null`.strip.presence,
+        build: `uname -v 2>/dev/null`.strip.presence,
+        kernel_version: `uname -a 2>/dev/null`.strip.presence,
+        machine: RbConfig::CONFIG["host_cpu"]
+      }.compact
+    rescue
+      { name: RbConfig::CONFIG["host_os"] }
+    end
+
+    def self.collect_runtime_context
+      {
+        name: "ruby",
+        version: RUBY_DESCRIPTION
+      }
+    end
+
+    def self.collect_modules
+      return {} unless defined?(Bundler)
+
+      Bundler.load.specs.each_with_object({}) do |spec, h|
+        h[spec.name] = spec.version.to_s
+      end
+    rescue
+      {}
+    end
+
+    def self.collect_server_name
+      Socket.gethostname
+    rescue
+      "unknown"
+    end
+
+    def self.collect_release
+      # Try to get from git if available
+      if File.exist?(".git")
+        `git rev-parse HEAD 2>/dev/null`.strip.presence
+      end
+    rescue
+      nil
+    end
+
+    def to_contexts
+      {
+        os: os,
+        runtime: runtime
+      }
+    end
+  end
+end

data/lib/lapsoss/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Lapsoss
-  VERSION = "0.4.5"
+  VERSION = "0.4.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lapsoss
 version: !ruby/object:Gem::Version
-  version: 0.4.5
+  version: 0.4.6
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -255,6 +255,7 @@ files:
 - lib/lapsoss/registry.rb
 - lib/lapsoss/release_tracker.rb
 - lib/lapsoss/router.rb
+- lib/lapsoss/runtime_context.rb
 - lib/lapsoss/sampling/base.rb
 - lib/lapsoss/sampling/rate_limiter.rb
 - lib/lapsoss/sampling/uniform_sampler.rb