lapsoss 0.1.0 → 0.2.0

data/lib/lapsoss/backtrace_processor.rb

@@ -1,9 +1,8 @@
 # frozen_string_literal: true
 
-require "set"
-require "active_support/cache"
-require "active_support/core_ext/numeric/time"
-require_relative "backtrace_frame"
+require 'active_support/cache'
+require 'active_support/core_ext/numeric/time'
+require_relative 'backtrace_frame'
 
 module Lapsoss
   class BacktraceProcessor
@@ -33,20 +32,20 @@ module Lapsoss
 
     def initialize(config = {})
       # Handle different config formats for backward compatibility
-      if config.respond_to?(:backtrace_context_lines)
-        # Configuration object passed
-        config_hash = {
-          context_lines: config.backtrace_context_lines,
-          max_frames: config.backtrace_max_frames,
-          enable_code_context: config.backtrace_enable_code_context,
-          in_app_patterns: config.backtrace_in_app_patterns,
-          exclude_patterns: config.backtrace_exclude_patterns,
-          strip_load_path: config.backtrace_strip_load_path
-        }
-      else
-        # Hash passed
-        config_hash = config
-      end
+      config_hash = if config.respond_to?(:backtrace_context_lines)
+                      # Configuration object passed
+                      {
+                        context_lines: config.backtrace_context_lines,
+                        max_frames: config.backtrace_max_frames,
+                        enable_code_context: config.backtrace_enable_code_context,
+                        in_app_patterns: config.backtrace_in_app_patterns,
+                        exclude_patterns: config.backtrace_exclude_patterns,
+                        strip_load_path: config.backtrace_strip_load_path
+                      }
+                    else
+                      # Hash passed
+                      config
+                    end
 
       @config = DEFAULT_CONFIG.merge(config_hash)
       @file_cache = ActiveSupport::Cache::MemoryStore.new(
@@ -77,7 +76,7 @@ module Lapsoss
     end
 
     # Backward compatibility alias
-    alias_method :process, :process_backtrace
+    alias process process_backtrace
 
     def process_exception_backtrace(exception, follow_cause: false)
       return [] unless exception&.backtrace
@@ -86,7 +85,7 @@ module Lapsoss
 
       # Add exception-specific context
       frames.each_with_index do |frame, index|
-        frame.define_singleton_method(:crash_frame?) { index == 0 }
+        frame.define_singleton_method(:crash_frame?) { index.zero? }
         frame.define_singleton_method(:exception_class) { exception.class.name }
       end
 
@@ -100,7 +99,7 @@ module Lapsoss
     end
 
     # Backward compatibility aliases
-    alias_method :process_exception, :process_exception_backtrace
+    alias process_exception process_exception_backtrace
 
     def clear_cache!
       @file_cache.clear
@@ -180,7 +179,7 @@ module Lapsoss
         file_cache: {
           # ActiveSupport::Cache::MemoryStore doesn't expose detailed stats
           # but we can provide basic info
-          type: "ActiveSupport::Cache::MemoryStore",
+          type: 'ActiveSupport::Cache::MemoryStore',
           configured_size: @file_cache.options[:size]
         },
         config: @config,
@@ -205,7 +204,7 @@ module Lapsoss
 
       # Convert to 0-based index
       line_index = line_number - 1
-      return nil if line_index < 0 || line_index >= lines.length
+      return nil if line_index.negative? || line_index >= lines.length
 
       # Calculate context range
       start_line = [0, line_index - context_lines].max
@@ -283,11 +282,11 @@ module Lapsoss
       head_frames = frames.first(head_count)
 
       # Get tail frames (original cause)
-      tail_frames = if tail_count > 0
-        frames.last(tail_count)
-      else
-        []
-      end
+      tail_frames = if tail_count.positive?
+                      frames.last(tail_count)
+                    else
+                      []
+                    end
 
       head_frames + tail_frames
     end
@@ -335,9 +334,7 @@ module Lapsoss
       paths << Dir.pwd
 
       # Add gem paths
-      if defined?(Gem)
-        paths.concat(Gem.path.map { |p| File.join(p, 'gems') })
-      end
+      paths.concat(Gem.path.map { |p| File.join(p, 'gems') }) if defined?(Gem)
 
       # Sort by length (longest first) for better matching
       paths.uniq.sort_by(&:length).reverse

data/lib/lapsoss/client.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require "concurrent"
-require_relative "scope"
-require_relative "current"
+require 'concurrent'
+require_relative 'scope'
+require_relative 'current'
 
 module Lapsoss
   class Client
@@ -70,8 +70,6 @@ module Lapsoss
 
     private
 
-
-
     def capture_event(event)
       # Apply pipeline processing if enabled
       if @configuration.enable_pipeline && @configuration.pipeline
@@ -87,7 +85,7 @@ module Lapsoss
       else
         Router.process_event(event)
       end
-    rescue => e
+    rescue StandardError => e
       handle_capture_error(e)
     end
 

data/lib/lapsoss/configuration.rb

@@ -1,26 +1,23 @@
 # frozen_string_literal: true
 
-require_relative "validators"
-require "active_support/configurable"
+require_relative 'validators'
+require 'active_support/configurable'
 
 module Lapsoss
   class Configuration
     include Validators
     include ActiveSupport::Configurable
 
-    attr_accessor :async, :logger, :enabled, :release,
+    attr_accessor :async, :logger, :enabled, :release, :debug,
                   :scrub_fields, :scrub_all, :whitelist_fields, :randomize_scrub_length,
                   :transport_jitter, :fingerprint_patterns,
                   :normalize_fingerprint_paths, :normalize_fingerprint_ids, :fingerprint_include_environment,
                   :backtrace_context_lines, :backtrace_in_app_patterns, :backtrace_exclude_patterns,
                   :backtrace_strip_load_path, :backtrace_max_frames, :backtrace_enable_code_context,
                   :enable_pipeline, :pipeline_builder, :sampling_strategy,
-                  :skip_rails_cache_errors
-    attr_reader :fingerprint_callback
-    attr_reader :environment, :before_send, :sample_rate, :error_handler,
-                :transport_timeout, :transport_max_retries, :transport_initial_backoff,
-                :transport_max_backoff, :transport_backoff_multiplier, :transport_ssl_verify
-    attr_reader :default_context, :adapter_configs
+                  :skip_rails_cache_errors, :force_sync_http
+    attr_reader :fingerprint_callback, :environment, :before_send, :sample_rate, :error_handler, :transport_timeout,
+                :transport_max_retries, :transport_initial_backoff, :transport_max_backoff, :transport_backoff_multiplier, :transport_ssl_verify, :default_context, :adapter_configs
 
     def initialize
       @adapter_configs = {}
@@ -29,11 +26,12 @@ module Lapsoss
       @environment = nil
       @enabled = true
       @release = nil
+      @debug = false
       @before_send = nil
       @sample_rate = 1.0
       @default_context = {}
       @error_handler = nil
-      @scrub_fields = nil  # Will use defaults from Scrubber
+      @scrub_fields = nil # Will use defaults from Scrubber
       @scrub_all = false
       @whitelist_fields = []
       @randomize_scrub_length = false
@@ -47,7 +45,7 @@ module Lapsoss
       @transport_ssl_verify = true
       # Fingerprinting settings
       @fingerprint_callback = nil
-      @fingerprint_patterns = nil  # Will use defaults from Fingerprinter
+      @fingerprint_patterns = nil # Will use defaults from Fingerprinter
       @normalize_fingerprint_paths = true
       @normalize_fingerprint_ids = true
       @fingerprint_include_environment = false
@@ -64,9 +62,10 @@ module Lapsoss
       @sampling_strategy = nil
       # Rails error filtering
       @skip_rails_cache_errors = true
+      # HTTP client settings
+      @force_sync_http = false
     end
 
-
     # Register a named adapter configuration
     #
     # @param name [Symbol] Unique name for this adapter instance
@@ -99,7 +98,6 @@ module Lapsoss
       register_adapter(name, :bugsnag, **settings)
     end
 
-
     # Convenience method for Rollbar
     def use_rollbar(name: :rollbar, **settings)
       register_adapter(name, :rollbar, **settings)
@@ -164,9 +162,17 @@ module Lapsoss
       initialize
     end
 
+    def debug?
+      @debug
+    end
+
+    def async?
+      @async
+    end
+
     # Pipeline configuration
-    def configure_pipeline(&block)
-      require_relative "pipeline"
+    def configure_pipeline
+      require_relative 'pipeline'
       @pipeline_builder = PipelineBuilder.new
       yield(@pipeline_builder) if block_given?
       @pipeline_builder
@@ -178,7 +184,7 @@ module Lapsoss
 
     # Sampling configuration
     def configure_sampling(strategy = nil, &block)
-      require_relative "sampling"
+      require_relative 'sampling'
 
       if strategy
         @sampling_strategy = strategy
@@ -188,7 +194,7 @@ module Lapsoss
     end
 
     def create_sampling_strategy
-      require_relative "sampling"
+      require_relative 'sampling'
 
       case @sampling_strategy
       when Symbol
@@ -213,85 +219,86 @@ module Lapsoss
 
     # Validation and setter overrides
     def sample_rate=(value)
-      validate_sample_rate!(value, "sample_rate") if value
+      validate_sample_rate!(value, 'sample_rate') if value
       @sample_rate = value
     end
 
     def before_send=(value)
-      validate_callable!(value, "before_send")
+      validate_callable!(value, 'before_send')
       @before_send = value
     end
 
     def error_handler=(value)
-      validate_callable!(value, "error_handler")
+      validate_callable!(value, 'error_handler')
       @error_handler = value
     end
 
     def environment=(value)
-      validate_environment!(value, "environment") if value
+      validate_environment!(value, 'environment') if value
       @environment = value&.to_s
     end
 
     def transport_timeout=(value)
-      validate_timeout!(value, "transport_timeout") if value
+      validate_timeout!(value, 'transport_timeout') if value
       @transport_timeout = value
     end
 
     def transport_max_retries=(value)
-      validate_retries!(value, "transport_max_retries") if value
+      validate_retries!(value, 'transport_max_retries') if value
       @transport_max_retries = value
     end
 
     def transport_initial_backoff=(value)
-      validate_timeout!(value, "transport_initial_backoff") if value
+      validate_timeout!(value, 'transport_initial_backoff') if value
       @transport_initial_backoff = value
     end
 
     def transport_max_backoff=(value)
-      validate_timeout!(value, "transport_max_backoff") if value
+      validate_timeout!(value, 'transport_max_backoff') if value
       @transport_max_backoff = value
     end
 
     def transport_backoff_multiplier=(value)
       if value
-        validate_type!(value, [Numeric], "transport_backoff_multiplier")
-        validate_numeric_range!(value, 1.0..10.0, "transport_backoff_multiplier")
+        validate_type!(value, [Numeric], 'transport_backoff_multiplier')
+        validate_numeric_range!(value, 1.0..10.0, 'transport_backoff_multiplier')
       end
       @transport_backoff_multiplier = value
     end
 
     def transport_ssl_verify=(value)
-      validate_boolean!(value, "transport_ssl_verify") if value
+      validate_boolean!(value, 'transport_ssl_verify') if value
       @transport_ssl_verify = value
     end
 
     def fingerprint_callback=(value)
-      validate_callable!(value, "fingerprint_callback")
+      validate_callable!(value, 'fingerprint_callback')
       @fingerprint_callback = value
     end
 
     # Configuration validation
     def validate!
-      validate_sample_rate!(@sample_rate, "sample_rate") if @sample_rate
-      validate_callable!(@before_send, "before_send")
-      validate_callable!(@error_handler, "error_handler")
-      validate_callable!(@fingerprint_callback, "fingerprint_callback")
-      validate_environment!(@environment, "environment") if @environment
+      validate_sample_rate!(@sample_rate, 'sample_rate') if @sample_rate
+      validate_callable!(@before_send, 'before_send')
+      validate_callable!(@error_handler, 'error_handler')
+      validate_callable!(@fingerprint_callback, 'fingerprint_callback')
+      validate_environment!(@environment, 'environment') if @environment
 
       # Validate transport settings
-      validate_timeout!(@transport_timeout, "transport_timeout")
-      validate_retries!(@transport_max_retries, "transport_max_retries")
-      validate_timeout!(@transport_initial_backoff, "transport_initial_backoff")
-      validate_timeout!(@transport_max_backoff, "transport_max_backoff")
+      validate_timeout!(@transport_timeout, 'transport_timeout')
+      validate_retries!(@transport_max_retries, 'transport_max_retries')
+      validate_timeout!(@transport_initial_backoff, 'transport_initial_backoff')
+      validate_timeout!(@transport_max_backoff, 'transport_max_backoff')
 
       if @transport_backoff_multiplier
-        validate_type!(@transport_backoff_multiplier, [Numeric], "transport_backoff_multiplier")
-        validate_numeric_range!(@transport_backoff_multiplier, 1.0..10.0, "transport_backoff_multiplier")
+        validate_type!(@transport_backoff_multiplier, [Numeric], 'transport_backoff_multiplier')
+        validate_numeric_range!(@transport_backoff_multiplier, 1.0..10.0, 'transport_backoff_multiplier')
       end
 
       # Validate that initial backoff is less than max backoff
       if @transport_initial_backoff && @transport_max_backoff && @transport_initial_backoff > @transport_max_backoff
-        raise ValidationError, "transport_initial_backoff (#{@transport_initial_backoff}) must be less than transport_max_backoff (#{@transport_max_backoff})"
+        raise ValidationError,
+              "transport_initial_backoff (#{@transport_initial_backoff}) must be less than transport_max_backoff (#{@transport_max_backoff})"
       end
 
       # Validate adapter configurations

data/lib/lapsoss/current.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "active_support/all"
+require 'active_support/all'
 
 module Lapsoss
   class Current < ActiveSupport::CurrentAttributes

data/lib/lapsoss/event.rb

@@ -1,17 +1,18 @@
 # frozen_string_literal: true
 
-require_relative "scrubber"
-require_relative "fingerprinter"
-require_relative "backtrace_processor"
+require_relative 'scrubber'
+require_relative 'fingerprinter'
+require_relative 'backtrace_processor'
 
 module Lapsoss
   class Event
-    attr_accessor :type, :timestamp, :level, :message, :exception, :context, :environment, :fingerprint, :backtrace_frames
+    attr_accessor :type, :timestamp, :level, :message, :exception, :context, :environment, :fingerprint,
+                  :backtrace_frames
 
     def initialize(type:, level: :info, **attributes)
       @type = type
       @level = level
-      @timestamp = Time.now
+      @timestamp = Time.zone.now
       @context = {}
       @environment = Lapsoss.configuration.environment
 

data/lib/lapsoss/{exclusions.rb → exclusion_filter.rb}

@@ -141,8 +141,8 @@ module Lapsoss
       return false if @excluded_environments.empty?
 
       environment = event.context[:environment] ||
-                   event.context.dig(:tags, :environment) ||
-                   Lapsoss.configuration.environment
+                    event.context.dig(:tags, :environment) ||
+                    Lapsoss.configuration.environment
 
       return false unless environment
 
@@ -160,12 +160,12 @@ module Lapsoss
       {
         excluded_exceptions: [
           # Test-related exceptions
-          "RSpec::Expectations::ExpectationNotMetError",
-          "Minitest::Assertion",
+          'RSpec::Expectations::ExpectationNotMetError',
+          'Minitest::Assertion',
 
           # Development tools
-          "Pry::CommandError",
-          "Byebug::CommandError"
+          'Pry::CommandError',
+          'Byebug::CommandError'
         ],
         excluded_patterns: [
           /test/i,
@@ -181,19 +181,19 @@ module Lapsoss
       {
         excluded_exceptions: [
           # Common Rails exceptions that are usually not actionable
-          "ActionController::RoutingError",
-          "ActionController::UnknownFormat",
-          "ActionController::BadRequest",
-          "ActionController::ParameterMissing",
+          'ActionController::RoutingError',
+          'ActionController::UnknownFormat',
+          'ActionController::BadRequest',
+          'ActionController::ParameterMissing',
 
           # ActiveRecord exceptions for common user errors
-          "ActiveRecord::RecordNotFound",
-          "ActiveRecord::RecordInvalid",
+          'ActiveRecord::RecordNotFound',
+          'ActiveRecord::RecordInvalid',
 
           # Network timeouts that are expected
-          "Net::ReadTimeout",
-          "Net::OpenTimeout",
-          "Timeout::Error"
+          'Net::ReadTimeout',
+          'Net::OpenTimeout',
+          'Timeout::Error'
         ],
         excluded_patterns: [
           # Bot and crawler patterns
@@ -215,10 +215,10 @@ module Lapsoss
         ],
         excluded_messages: [
           # Common spam/attack messages
-          "No route matches",
-          "Invalid authenticity token",
-          "Forbidden",
-          "Unauthorized"
+          'No route matches',
+          'Invalid authenticity token',
+          'Forbidden',
+          'Unauthorized'
         ]
       }
     end
@@ -227,8 +227,8 @@ module Lapsoss
       {
         excluded_exceptions: [
           # Test data related errors
-          "ActiveRecord::RecordNotFound",
-          "ArgumentError"
+          'ActiveRecord::RecordNotFound',
+          'ArgumentError'
         ],
         excluded_patterns: [
           /test/i,
@@ -267,15 +267,15 @@ module Lapsoss
           /onerror=/i
         ],
         excluded_messages: [
-          "Invalid authenticity token",
-          "Forbidden",
-          "Unauthorized",
-          "Access denied"
+          'Invalid authenticity token',
+          'Forbidden',
+          'Unauthorized',
+          'Access denied'
         ],
         custom_filters: [
           # Exclude requests from known bot user agents
           lambda do |event|
-            user_agent = event.context.dig(:request, :headers, "User-Agent")
+            user_agent = event.context.dig(:request, :headers, 'User-Agent')
             return false unless user_agent
 
             bot_patterns = [
@@ -297,14 +297,14 @@ module Lapsoss
       {
         excluded_exceptions: [
           # Timeout exceptions that are expected under load
-          "Net::ReadTimeout",
-          "Net::OpenTimeout",
-          "Timeout::Error",
-          "Redis::TimeoutError",
+          'Net::ReadTimeout',
+          'Net::OpenTimeout',
+          'Timeout::Error',
+          'Redis::TimeoutError',
 
           # Memory and resource limits
-          "NoMemoryError",
-          "SystemStackError"
+          'NoMemoryError',
+          'SystemStackError'
         ],
         excluded_patterns: [
           /timeout/i,
@@ -315,7 +315,7 @@ module Lapsoss
         custom_filters: [
           # Exclude high-frequency errors during peak times
           lambda do |event|
-            now = Time.now
+            now = Time.zone.now
             peak_hours = (9..17).cover?(now.hour) && (1..5).cover?(now.wday)
 
             if peak_hours
@@ -334,11 +334,11 @@ module Lapsoss
       {
         excluded_exceptions: [
           # User input validation errors
-          "ActiveModel::ValidationError",
-          "ActiveRecord::RecordInvalid",
-          "ActionController::ParameterMissing",
-          "ArgumentError",
-          "TypeError"
+          'ActiveModel::ValidationError',
+          'ActiveRecord::RecordInvalid',
+          'ActionController::ParameterMissing',
+          'ArgumentError',
+          'TypeError'
         ],
         excluded_patterns: [
           /validation/i,
@@ -373,15 +373,15 @@ module Lapsoss
 
       presets.each do |preset|
         config = case preset
-        when :development then development
-        when :production then production
-        when :staging then staging
-        when :security_focused then security_focused
-        when :performance_focused then performance_focused
-        when :user_error_focused then user_error_focused
-        when Hash then preset
-        else raise ArgumentError, "Unknown preset: #{preset}"
-        end
+                 when :development then development
+                 when :production then production
+                 when :staging then staging
+                 when :security_focused then security_focused
+                 when :performance_focused then performance_focused
+                 when :user_error_focused then user_error_focused
+                 when Hash then preset
+                 else raise ArgumentError, "Unknown preset: #{preset}"
+                 end
 
         combined_config[:excluded_exceptions].concat(config[:excluded_exceptions] || [])
         combined_config[:excluded_patterns].concat(config[:excluded_patterns] || [])
@@ -404,15 +404,15 @@ module Lapsoss
   module ExclusionConfiguration
     def self.configure_exclusions(config, preset: nil, **custom_config)
       exclusion_config = if preset
-        case preset
-        when Array
-          ExclusionPresets.combined(preset)
-        else
-          ExclusionPresets.send(preset)
-        end
-      else
-        {}
-      end
+                           case preset
+                           when Array
+                             ExclusionPresets.combined(preset)
+                           else
+                             ExclusionPresets.send(preset)
+                           end
+                         else
+                           {}
+                         end
 
       # Merge custom configuration
       exclusion_config.merge!(custom_config)