appsignal 4.0.7-java → 4.0.9-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 17a0e4b7548920603d0dcba11762fc6043e263138903671fd7bc7da41d587ba4
-  data.tar.gz: 270076b4c45a2f5581f7ed8a6fe952cabefd52779f1f033c5c1e1212757772c2
+  metadata.gz: bbb72b56837756abd3ec4d56855208a44ced0aabac240846bc13e685365ff79e
+  data.tar.gz: bca5d410c1a2fb91812d2cc63b19a04b0db6cafd18c368a1ece930915bbf0a66
 SHA512:
-  metadata.gz: 17b008d3d0198db02366d4446a718e44da22bd8bef6fb0c373710fdd8219e56864d10958f8d5d0d6960ee3f7c2e65743543b6d509e20f0ca0d2c861a750b0c16
-  data.tar.gz: 5f6284b47a65d258bfc567f6d7ba68c91458251bfc39d74709644324c8f9e277657d42eca93077509a837dfba3fe412385b2d853e73aba7c1f0863df5356efc0
+  metadata.gz: cd24c802a14d24e154f99493a242c815b85ab96c7049d589d0d1e26eb097d53e91eecc7732e97b8f2ea7415edef21df79bfbcbe1015f7cb5823f292ff30f9939
+  data.tar.gz: 5e3d0d285a6ebf6a7b8beae27fdaceec6365aa016fa1181e1572f42054fced494771eac6fd62434efb5cd4dae7ee7ca542aa1b6ac5d680139ec7d24dfd238c55

data/CHANGELOG.md

@@ -1,12 +1,42 @@
 # AppSignal for Ruby gem Changelog
 
+## 4.0.9
+
+_Published on 2024-09-17._
+
+### Changed
+
+- Add the logger gem as a dependency. This fixes the deprecation warning on Ruby 3.3. (patch [8c1d577e](https://github.com/appsignal/appsignal-ruby/commit/8c1d577e4790185db887d49577cedc7d614d8d98))
+- Do not report errors caused by `Errno::EPIPE` (broken pipe errors) when instrumenting response bodies, to avoid reporting errors that cannot be fixed by the application. (patch [1fdccba4](https://github.com/appsignal/appsignal-ruby/commit/1fdccba4ceeb8f9bb13ae077019b2c1f7d9d4fe4))
+- Normalize Rack and Rails `UploadedFile` objects. Instead of displaying the Ruby class name, it will now show object details like the filename and content type.
+
+  ```
+  # Before
+  #<Rack::Multipart::UploadedFile>
+  #<ActionDispatch::Http::UploadedFile>
+
+  # After
+  #<Rack::Multipart::UploadedFile original_filename: "uploaded_file.txt", content_type: "text/plain">
+  #<ActionDispatch::Http::UploadedFile original_filename: "uploaded_file.txt", content_type: "text/plain">
+  ```
+
+  (patch [bb50c933](https://github.com/appsignal/appsignal-ruby/commit/bb50c93387eafebe043b0e7f4083c95556b93136))
+
+## 4.0.8
+
+_Published on 2024-09-13._
+
+### Fixed
+
+- Fix a `ThreadError` from being raised on process exit when `Appsignal.stop` is called from a `Signal.trap` block, like when Puma shuts down in clustered mode. (patch [32323ded](https://github.com/appsignal/appsignal-ruby/commit/32323ded277d4764ea1bd0d0dab02bef3de40ccb))
+
 ## 4.0.7
 
 _Published on 2024-09-12._
 
 ### Changed
 
-- Format the Date and Time objects in a human-friendly way. Previously, dates and times stored in sample data, like session data, would be shown as `#<Date>` and `#<Time>`. Now they will show as `#<Date: 2024-09-11>` and `#<Time: Time: 2024-09-12T13:14:15+02:00>` (UTC offset may be different for your time objects depending on the server setting). (patch [8f516484](https://github.com/appsignal/appsignal-ruby/commit/8f516484d249f43ffadcf15a67bbab48f827eff6))
+- Format the Date and Time objects in a human-friendly way. Previously, dates and times stored in sample data, like session data, would be shown as `#<Date>` and `#<Time>`. Now they will show as `#<Date: 2024-09-11>` and `#<Time: 2024-09-12T13:14:15+02:00>` (UTC offset may be different for your time objects depending on the server setting). (patch [8f516484](https://github.com/appsignal/appsignal-ruby/commit/8f516484d249f43ffadcf15a67bbab48f827eff6))
 
 ### Removed
 

data/appsignal.gemspec

@@ -17,8 +17,8 @@ IGNORED_PATHS = [
   ".yardopts",
   "benchmark.rake",
   "mono.yml",
-  "rubocop.yml",
-  "rubocop_todo.yml"
+  ".rubocop.yml",
+  ".rubocop_todo.yml"
 ].freeze
 
 Gem::Specification.new do |gem| # rubocop:disable Metrics/BlockLength
@@ -56,6 +56,7 @@ Gem::Specification.new do |gem| # rubocop:disable Metrics/BlockLength
     "source_code_uri" => "https://github.com/appsignal/appsignal-ruby"
   }
 
+  gem.add_dependency "logger"
   gem.add_dependency "rack"
 
   gem.add_development_dependency "pry"

data/lib/appsignal/rack/body_wrapper.rb

@@ -54,7 +54,7 @@ module Appsignal
       rescue *IGNORED_ERRORS # Do not report
         raise
       rescue Exception => error # rubocop:disable Lint/RescueException
-        @transaction.set_error(error)
+        appsignal_report_error(error)
         raise error
       end
 
@@ -72,6 +72,19 @@ module Appsignal
         @body.__send__(method_name, *args, &block)
       end
       ruby2_keywords(:method_missing) if respond_to?(:ruby2_keywords, true)
+
+      private
+
+      def appsignal_report_error(error)
+        @transaction.set_error(error) if appsignal_accepted_error?(error)
+      end
+
+      def appsignal_accepted_error?(error)
+        return true unless error.cause
+        return false if IGNORED_ERRORS.include?(error.cause.class)
+
+        appsignal_accepted_error?(error.cause)
+      end
     end
 
     # The standard Rack body wrapper which exposes "each" for iterating
@@ -97,7 +110,7 @@ module Appsignal
       rescue *IGNORED_ERRORS # Do not report
         raise
       rescue Exception => error # rubocop:disable Lint/RescueException
-        @transaction.set_error(error)
+        appsignal_report_error(error)
         raise error
       end
     end
@@ -118,7 +131,7 @@ module Appsignal
       rescue *IGNORED_ERRORS # Do not report
         raise
       rescue Exception => error # rubocop:disable Lint/RescueException
-        @transaction.set_error(error)
+        appsignal_report_error(error)
         raise error
       end
     end
@@ -144,7 +157,7 @@ module Appsignal
       rescue *IGNORED_ERRORS # Do not report
         raise
       rescue Exception => error # rubocop:disable Lint/RescueException
-        @transaction.set_error(error)
+        appsignal_report_error(error)
         raise error
       end
     end
@@ -162,7 +175,7 @@ module Appsignal
       rescue *IGNORED_ERRORS # Do not report
         raise
       rescue Exception => error # rubocop:disable Lint/RescueException
-        @transaction.set_error(error)
+        appsignal_report_error(error)
         raise error
       end
     end

data/lib/appsignal/transaction.rb

@@ -707,7 +707,7 @@ module Appsignal
       return unless Appsignal.config[:send_params]
 
       filter_keys = Appsignal.config[:filter_parameters] || []
-      Appsignal::Utils::HashSanitizer.sanitize params, filter_keys
+      Appsignal::Utils::SampleDataSanitizer.sanitize(params, filter_keys)
     end
 
     def session_data
@@ -720,7 +720,8 @@ module Appsignal
 
     # Returns sanitized session data.
     #
-    # The session data is sanitized by the {Appsignal::Utils::HashSanitizer}.
+    # The session data is sanitized by the
+    # {Appsignal::Utils::SampleDataSanitizer}.
     #
     # @return [nil] if `:send_session_data` config is set to `false`.
     # @return [nil] if the {#request} object doesn't respond to `#session`.
@@ -729,8 +730,9 @@ module Appsignal
     def sanitized_session_data
       return unless Appsignal.config[:send_session_data]
 
-      Appsignal::Utils::HashSanitizer.sanitize(
-        session_data, Appsignal.config[:filter_session_data]
+      Appsignal::Utils::SampleDataSanitizer.sanitize(
+        session_data,
+        Appsignal.config[:filter_session_data]
       )
     end
 

data/lib/appsignal/utils/data.rb

@@ -25,7 +25,7 @@ module Appsignal
               # An Integer too big for C-lang longs to fit
               bigint = 1 << 63
               if value >= bigint
-                map.set_string(key, "bigint:#{value}")
+                map.set_string(key, map_bigint(value))
               else
                 map.set_integer(key, value)
               end
@@ -56,7 +56,7 @@ module Appsignal
               # An Integer too big for C-lang longs to fit
               bigint = 1 << 63
               if value >= bigint
-                array.append_string("bigint:#{value}")
+                array.append_string(map_bigint(value))
               else
                 array.append_integer(value)
               end
@@ -76,6 +76,10 @@ module Appsignal
           end
           array
         end
+
+        def map_bigint(value)
+          "bigint:#{value}"
+        end
       end
     end
   end

data/lib/appsignal/utils/{hash_sanitizer.rb → sample_data_sanitizer.rb}

@@ -2,7 +2,7 @@
 
 module Appsignal
   module Utils
-    class HashSanitizer
+    class SampleDataSanitizer
       FILTERED = "[FILTERED]"
       RECURSIVE = "[RECURSIVE VALUE]"
 
@@ -26,7 +26,21 @@ module Appsignal
           when Date
             "#<Date: #{value.iso8601}>"
           else
-            inspected(value)
+            if defined?(::Rack::Multipart::UploadedFile) &&
+                value.is_a?(::Rack::Multipart::UploadedFile)
+              "#<Rack::Multipart::UploadedFile " \
+                "original_filename: #{value.original_filename.inspect}, " \
+                "content_type: #{value.content_type.inspect}" \
+                ">"
+            elsif defined?(::ActionDispatch::Http::UploadedFile) &&
+                value.is_a?(::ActionDispatch::Http::UploadedFile)
+              "#<ActionDispatch::Http::UploadedFile " \
+                "original_filename: #{value.original_filename.inspect}, " \
+                "content_type: #{value.content_type.inspect}" \
+                ">"
+            else
+              inspected(value)
+            end
           end
         end
 

data/lib/appsignal/utils.rb

@@ -9,7 +9,7 @@ end
 require "appsignal/utils/integration_memory_logger"
 require "appsignal/utils/stdout_and_logger_message"
 require "appsignal/utils/data"
-require "appsignal/utils/hash_sanitizer"
+require "appsignal/utils/sample_data_sanitizer"
 require "appsignal/utils/integration_logger"
 require "appsignal/utils/json"
 require "appsignal/utils/ndjson"

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Appsignal
-  VERSION = "4.0.7"
+  VERSION = "4.0.9"
 end

data/lib/appsignal.rb

@@ -157,14 +157,16 @@ module Appsignal
     # @return [void]
     # @since 1.0.0
     def stop(called_by = nil)
-      if called_by
-        internal_logger.debug("Stopping AppSignal (#{called_by})")
-      else
-        internal_logger.debug("Stopping AppSignal")
-      end
-      Appsignal::Extension.stop
-      Appsignal::Probes.stop
-      Appsignal::CheckIn.stop
+      Thread.new do
+        if called_by
+          internal_logger.debug("Stopping AppSignal (#{called_by})")
+        else
+          internal_logger.debug("Stopping AppSignal")
+        end
+        Appsignal::Extension.stop
+        Appsignal::Probes.stop
+        Appsignal::CheckIn.stop
+      end.join
     end
 
     # Configure the AppSignal Ruby gem using a DSL.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 4.0.7
+  version: 4.0.9
 platform: java
 authors:
 - Robert Beekman
@@ -10,8 +10,22 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-12 00:00:00.000000000 Z
+date: 2024-09-17 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: logger
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
@@ -147,8 +161,6 @@ extensions:
 - ext/Rakefile
 extra_rdoc_files: []
 files:
-- ".rubocop.yml"
-- ".rubocop_todo.yml"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
@@ -274,13 +286,13 @@ files:
 - lib/appsignal/transmitter.rb
 - lib/appsignal/utils.rb
 - lib/appsignal/utils/data.rb
-- lib/appsignal/utils/hash_sanitizer.rb
 - lib/appsignal/utils/integration_logger.rb
 - lib/appsignal/utils/integration_memory_logger.rb
 - lib/appsignal/utils/json.rb
 - lib/appsignal/utils/ndjson.rb
 - lib/appsignal/utils/query_params_sanitizer.rb
 - lib/appsignal/utils/rails_helper.rb
+- lib/appsignal/utils/sample_data_sanitizer.rb
 - lib/appsignal/utils/stdout_and_logger_message.rb
 - lib/appsignal/version.rb
 - lib/puma/plugin/appsignal.rb

data/.rubocop.yml

@@ -1,133 +0,0 @@
-inherit_from: .rubocop_todo.yml
-
-AllCops:
-  TargetRubyVersion: 2.7
-  NewCops: enable
-  Include:
-    - "**/*.rb"
-    - "**/*.cap"
-    - "**/*.rake"
-    - "**/Gemfile"
-    - "**/Rakefile"
-    - "appsignal.gemspec"
-  Exclude:
-    - "pkg/**/*"
-    - "tmp/**/*"
-    - "vendor/**/*"
-    - "spec/integration/diagnose/**/*"
-  DisplayCopNames: true
-  UseCache: true
-  CacheRootDirectory: ./tmp
-
-Style/RescueStandardError:
-  Enabled: false
-
-Style/Documentation:
-  Enabled: false
-
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
-
-Style/StringLiteralsInInterpolation:
-  EnforcedStyle: double_quotes
-
-Style/HashSyntax:
-  EnforcedStyle: hash_rockets
-
-Style/EmptyMethod:
-  EnforcedStyle: expanded
-
-Style/MissingRespondToMissing:
-  Exclude:
-    - "lib/appsignal/extension.rb"
-    - "lib/appsignal/transaction.rb"
-
-Style/TrailingUnderscoreVariable:
-  Enabled: false
-
-Style/Lambda:
-  EnforcedStyle: lambda
-
-Style/WordArray:
-  Enabled: false
-
-Style/FrozenStringLiteralComment:
-  Enabled: true
-  Exclude:
-    - "spec/**/*.rb"
-
-Style/NumericPredicate:
-  Enabled: false
-
-Style/SymbolArray:
-  EnforcedStyle: brackets
-
-Style/RedundantConstantBase:
-  Enabled: false
-
-Lint/ConstantDefinitionInBlock:
-  Exclude:
-    - "spec/**/*.rb"
-
-Lint/EmptyClass:
-  Exclude:
-    - "spec/**/*.rb"
-
-Lint/EmptyFile:
-  Exclude:
-    - "spec/**/*.rb"
-
-Layout/HashAlignment:
-  EnforcedLastArgumentHashStyle: ignore_implicit
-
-Layout/ArgumentAlignment:
-  EnforcedStyle: with_fixed_indentation
-
-Layout/LineContinuationLeadingSpace:
-  Enabled: false
-
-Layout/FirstArrayElementIndentation:
-  EnforcedStyle: consistent
-
-Layout/LineEndStringConcatenationIndentation:
-  EnforcedStyle: indented
-
-Layout/ParameterAlignment:
-  EnforcedStyle: with_fixed_indentation
-
-Layout/MultilineMethodCallIndentation:
-  EnforcedStyle: indented
-
-Layout/MultilineOperationIndentation:
-  EnforcedStyle: indented
-
-Layout/LineLength:
-  Max: 100
-
-Naming/FileName:
-  Exclude:
-    - "ext/Rakefile"
-
-Naming/AccessorMethodName:
-  Exclude:
-    - "lib/appsignal/helpers/instrumentation.rb"
-    - "lib/appsignal/transaction.rb"
-
-Naming/RescuedExceptionsVariableName:
-  Enabled: false
-
-Naming/VariableNumber:
-  Enabled: false
-
-Metrics/ModuleLength:
-  Enabled: false
-
-Metrics/ClassLength:
-  Enabled: false
-
-Metrics/BlockLength:
-  Exclude:
-    - "Rakefile"
-
-Gemspec/DevelopmentDependencies:
-  Enabled: false

data/.rubocop_todo.yml

@@ -1,84 +0,0 @@
-# This configuration was generated by
-# `rubocop --auto-gen-config`
-# on 2024-06-27 09:42:06 UTC using RuboCop version 1.64.1.
-# The point is for the user to remove these configuration records
-# one by one as the offenses are removed from the code base.
-# Note that changes in the inspected code, or installation of new
-# versions of RuboCop, may require this file to be generated again.
-
-# Offense count: 2
-# Configuration parameters: AllowedParentClasses.
-Lint/MissingSuper:
-  Exclude:
-    - 'lib/appsignal/extension.rb'
-    - 'lib/appsignal/logger.rb'
-
-# Offense count: 1
-Lint/StructNewOverride:
-  Exclude:
-    - 'spec/lib/appsignal/probes/sidekiq_spec.rb'
-
-# Offense count: 63
-# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
-Metrics/AbcSize:
-  Max: 44
-
-# Offense count: 6
-# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
-# AllowedMethods: refine
-Metrics/BlockLength:
-  Max: 31
-
-# Offense count: 21
-# Configuration parameters: AllowedMethods, AllowedPatterns.
-Metrics/CyclomaticComplexity:
-  Max: 11
-
-# Offense count: 139
-# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
-Metrics/MethodLength:
-  Max: 56
-
-# Offense count: 18
-# Configuration parameters: AllowedMethods, AllowedPatterns.
-Metrics/PerceivedComplexity:
-  Max: 13
-
-# Offense count: 2
-Security/Open:
-  Exclude:
-    - 'ext/base.rb'
-
-# Offense count: 2
-# This cop supports unsafe autocorrection (--autocorrect-all).
-Security/YAMLLoad:
-  Exclude:
-    - 'lib/appsignal/config.rb'
-    - 'lib/appsignal/integrations/sidekiq.rb'
-
-# Offense count: 7
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: prefer_alias, prefer_alias_method
-Style/Alias:
-  Exclude:
-    - 'lib/appsignal/helpers/instrumentation.rb'
-    - 'lib/appsignal/transaction.rb'
-
-# Offense count: 1
-Style/ClassVars:
-  Exclude:
-    - 'spec/lib/appsignal/event_formatter_spec.rb'
-
-# Offense count: 1
-Style/OpenStructUse:
-  Exclude:
-    - 'lib/appsignal/cli/install.rb'
-
-# Offense count: 2
-# Configuration parameters: AllowedMethods.
-# AllowedMethods: respond_to_missing?
-Style/OptionalBooleanParameter:
-  Exclude:
-    - 'lib/appsignal/integrations/delayed_job_plugin.rb'
-    - 'lib/appsignal/utils/query_params_sanitizer.rb'