appsignal 2.7.0.alpha.4 → 2.7.0.beta.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c61b262b517a8dc0923998d62be036fd6133c7cc
-  data.tar.gz: 3d84a9925bf05798201d91e3174823ffe6c793e6
+SHA256:
+  metadata.gz: b23b27d0c6cd268d86cc544d68ef947e365bbf6895cf84968f75ebd683d996f2
+  data.tar.gz: d7e8e9c06423301cc6ad0bd5a782e6059d0ac9373fd885425d114d75026c7b5b
 SHA512:
-  metadata.gz: 77170a310402131b77ad99f147c754ae328b3c0a688eee5306fe4d5468ea9f8eec51a04fdf3cf80cebf6374771fade3689155b5006832df84294c6a4324e6445
-  data.tar.gz: b60fd4f13ecc8c4c6c8cec97d5d0796c9a815abb56d3d5543950fd99ec14242e4cd4794eea53028b10be8bbd124216ac0b9e77cddf9eee54220f0d101c034eac
+  metadata.gz: 1cd7206a51c7e076fdef7ac3945d86074044271973946af3274cb9dd5aad1d0b756ac432a40b19224dd5eb5e59b1053bb0849f700f9795fa335dece2f0360566
+  data.tar.gz: 63e74bc3338a8de89237dec4b7c26c6dffce3d9d40cea768e1b94670db8bb45ea8160d019969d643c180e531369528657d7c3f31a6c4fbb88b79f754cf945367

data/.rubocop_todo.yml

@@ -1,12 +1,12 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2018-05-06 12:47:25 +0200 using RuboCop version 0.50.0.
+# on 2018-06-01 17:12:10 +0200 using RuboCop version 0.50.0.
 # 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: 45
+# Offense count: 44
 Metrics/AbcSize:
   Max: 56
 
@@ -28,13 +28,13 @@ Metrics/ClassLength:
 Metrics/CyclomaticComplexity:
   Max: 11
 
-# Offense count: 537
+# Offense count: 523
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
 Metrics/LineLength:
   Max: 152
 
-# Offense count: 95
+# Offense count: 94
 # Configuration parameters: CountComments.
 Metrics/MethodLength:
   Max: 60
@@ -108,10 +108,3 @@ Style/GuardClause:
     - 'lib/appsignal/event_formatter/moped/query_formatter.rb'
     - 'lib/appsignal/hooks.rb'
     - 'lib/appsignal/marker.rb'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-# Configuration parameters: MaxLineLength.
-Style/IfUnlessModifier:
-  Exclude:
-    - 'lib/appsignal.rb'

data/CHANGELOG.md

@@ -1,9 +1,11 @@
-# 2.7.0 Alpha 3
-- Fix reporting of memory host metrics for containerized hosts. PR #415
-- Detect Kubernetes containers as containers in probes-rs library.
-  Commit 60822aac24ccc394df073091c64f05096455942d.
+# 2.7.0 Beta 1
 - Detect Kubernetes containers as containers for `running_in_container`
   config option. Commit 60822aac24ccc394df073091c64f05096455942d.
+- Fix in memory logger initialization. PR #416
+- Organize classes in their own files. PR #417
+- Move tag value limit handling to extension. PR #418
+- Add working_directory_path config option. PR #421
+- Use doubles values in custom metrics functions. PR #422
 
 # 2.6.1
 - Remove request_headers warning and use sane default. PR #410

data/ext/agent.yml

@@ -1,64 +1,64 @@
 ---
-version: 94aaf32
+version: 64deb96
 triples:
   x86_64-darwin:
     static:
-      checksum: 1316f7c202a232ae691dc3745f2ec6ee824e01aa2a613e56a41979b19ba1ec8f
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-darwin-all-static.tar.gz
+      checksum: 1ab49acb93615b0c1e2fe9e48c179e0fd3fcadd391523940c322f154bd479a83
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-darwin-all-static.tar.gz
     dynamic:
-      checksum: b54ae98cfedfe82d7c4b6fbb05ec6d0310484db5706eeeb29a8cda006b5ea361
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-darwin-all-dynamic.tar.gz
+      checksum: 8b1acc7825c2fd024e4fed9bd4fe5485c3ba8dc903a00dbcd26d3ab27dc2e7ee
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-darwin-all-dynamic.tar.gz
   universal-darwin:
     static:
-      checksum: 1316f7c202a232ae691dc3745f2ec6ee824e01aa2a613e56a41979b19ba1ec8f
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-darwin-all-static.tar.gz
+      checksum: 1ab49acb93615b0c1e2fe9e48c179e0fd3fcadd391523940c322f154bd479a83
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-darwin-all-static.tar.gz
     dynamic:
-      checksum: b54ae98cfedfe82d7c4b6fbb05ec6d0310484db5706eeeb29a8cda006b5ea361
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-darwin-all-dynamic.tar.gz
+      checksum: 8b1acc7825c2fd024e4fed9bd4fe5485c3ba8dc903a00dbcd26d3ab27dc2e7ee
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-darwin-all-dynamic.tar.gz
   i686-linux:
     static:
-      checksum: c0c5c3ad646e500c2a4a78afadbf4828560e4b4f00901a726fd06af0e5047cdc
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-i686-linux-all-static.tar.gz
+      checksum: a27cf9191f70f13263662f6ca2555cada27ebb4384d6c981a534774b5e7def0e
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-i686-linux-all-static.tar.gz
     dynamic:
-      checksum: a6b98af9a1ac6d016b7021e73b76b457b6a1890114719fe6552eb46a371731ae
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-i686-linux-all-dynamic.tar.gz
+      checksum: 7a19dd1aabe68a454649155e99fd95293915d0019ba438366c9cce247cdb4bc1
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-i686-linux-all-dynamic.tar.gz
   x86-linux:
     static:
-      checksum: c0c5c3ad646e500c2a4a78afadbf4828560e4b4f00901a726fd06af0e5047cdc
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-i686-linux-all-static.tar.gz
+      checksum: a27cf9191f70f13263662f6ca2555cada27ebb4384d6c981a534774b5e7def0e
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-i686-linux-all-static.tar.gz
     dynamic:
-      checksum: a6b98af9a1ac6d016b7021e73b76b457b6a1890114719fe6552eb46a371731ae
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-i686-linux-all-dynamic.tar.gz
+      checksum: 7a19dd1aabe68a454649155e99fd95293915d0019ba438366c9cce247cdb4bc1
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-i686-linux-all-dynamic.tar.gz
   i686-linux-musl:
     static:
-      checksum: 05bfaa688ceef363523818b2d0d5d94db178b2596cca96d804932c129d8ede10
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-i686-linux-musl-all-static.tar.gz
+      checksum: 80fb5c2f36b7e8efcfc6914bb707688e248dd06c58b174d07e8a931f4bbf97ce
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-i686-linux-musl-all-static.tar.gz
   x86-linux-musl:
     static:
-      checksum: 05bfaa688ceef363523818b2d0d5d94db178b2596cca96d804932c129d8ede10
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-i686-linux-musl-all-static.tar.gz
+      checksum: 80fb5c2f36b7e8efcfc6914bb707688e248dd06c58b174d07e8a931f4bbf97ce
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-i686-linux-musl-all-static.tar.gz
   x86_64-linux:
     static:
-      checksum: db176c740e26db37d61df1f9e18d57127a1a5208495f02bf18ff2c9553b12d62
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-linux-all-static.tar.gz
+      checksum: 83befc3bb521d5e055eb92d5a5a0afca929bce8968ca6d0426fac6aeaec3df24
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-linux-all-static.tar.gz
     dynamic:
-      checksum: cb0f024f8a027d2a059c7d1a12fb98071e3c2da0ae6277ef5ac32fddf1e7ae8c
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-linux-all-dynamic.tar.gz
+      checksum: a6a8237df0fa0c8ff62ca3cd519e0494f01fc5e52f5ae9757c5fe375f451673a
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-linux-all-dynamic.tar.gz
   x86_64-linux-musl:
     static:
-      checksum: a3c5882bfa3ee6f9612449a8c90eb64791764f34d910c21fd436ce26cf232b4e
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-linux-musl-all-static.tar.gz
+      checksum: 3cb74695955c7bc809320fa966d7290fe3162926c239bb6a845b82ef7aec266e
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-linux-musl-all-static.tar.gz
   x86_64-freebsd:
     static:
-      checksum: ba8e6b26b76809a22753a6e6829be72c45a1c2a31c113e04d0fe09789ff63d98
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-freebsd-all-static.tar.gz
+      checksum: 7735718e2789c71f1135100ff4fab13cd85cbebe0ba2b87c5b555b30635bdcdc
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-freebsd-all-static.tar.gz
     dynamic:
-      checksum: 52b17d8504aa335a35c9a84cde5762096b2e94ee2c9d622397e240ffa01cf5cd
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-freebsd-all-dynamic.tar.gz
+      checksum: '059ee407610bd4154a8a94bb029d0451758bfb995b530ab5142e4fd3a1a59030'
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-freebsd-all-dynamic.tar.gz
   amd64-freebsd:
     static:
-      checksum: ba8e6b26b76809a22753a6e6829be72c45a1c2a31c113e04d0fe09789ff63d98
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-freebsd-all-static.tar.gz
+      checksum: 7735718e2789c71f1135100ff4fab13cd85cbebe0ba2b87c5b555b30635bdcdc
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-freebsd-all-static.tar.gz
     dynamic:
-      checksum: 52b17d8504aa335a35c9a84cde5762096b2e94ee2c9d622397e240ffa01cf5cd
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/94aaf32/appsignal-x86_64-freebsd-all-dynamic.tar.gz
+      checksum: '059ee407610bd4154a8a94bb029d0451758bfb995b530ab5142e4fd3a1a59030'
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/64deb96/appsignal-x86_64-freebsd-all-dynamic.tar.gz

data/ext/appsignal_extension.c

@@ -588,14 +588,14 @@ static VALUE increment_counter(VALUE self, VALUE key, VALUE count, VALUE tags) {
   appsignal_data_t* tags_data;
 
   Check_Type(key, T_STRING);
-  Check_Type(count, T_FIXNUM);
+  Check_Type(count, T_FLOAT);
   Check_Type(tags, RUBY_T_DATA);
 
   Data_Get_Struct(tags, appsignal_data_t, tags_data);
 
   appsignal_increment_counter(
       make_appsignal_string(key),
-      FIX2INT(count),
+      NUM2DBL(count),
       tags_data
   );
   return Qnil;

data/lib/appsignal.rb

@@ -600,7 +600,11 @@ module Appsignal
     end
 
     def set_gauge(key, value, tags = {})
-      Appsignal::Extension.set_gauge(key.to_s, value.to_f, Appsignal::Utils.data_generate(tags))
+      Appsignal::Extension.set_gauge(
+        key.to_s,
+        value.to_f,
+        Appsignal::Utils::Data.generate(tags)
+      )
     rescue RangeError
       Appsignal.logger.warn("Gauge value #{value} for key '#{key}' is too big")
     end
@@ -617,14 +621,22 @@ module Appsignal
       Appsignal.logger.warn("Process gauge value #{value} for key '#{key}' is too big")
     end
 
-    def increment_counter(key, value = 1, tags = {})
-      Appsignal::Extension.increment_counter(key.to_s, value, Appsignal::Utils.data_generate(tags))
+    def increment_counter(key, value = 1.0, tags = {})
+      Appsignal::Extension.increment_counter(
+        key.to_s,
+        value.to_f,
+        Appsignal::Utils::Data.generate(tags)
+      )
     rescue RangeError
       Appsignal.logger.warn("Counter value #{value} for key '#{key}' is too big")
     end
 
     def add_distribution_value(key, value, tags = {})
-      Appsignal::Extension.add_distribution_value(key.to_s, value.to_f, Appsignal::Utils.data_generate(tags))
+      Appsignal::Extension.add_distribution_value(
+        key.to_s,
+        value.to_f,
+        Appsignal::Utils::Data.generate(tags)
+      )
     rescue RangeError
       Appsignal.logger.warn("Distribution value #{value} for key '#{key}' is too big")
     end
@@ -682,9 +694,7 @@ module Appsignal
           Logger::INFO
         end
 
-      if in_memory_log
-        logger << in_memory_log.string
-      end
+      logger << @in_memory_log.string if @in_memory_log
 
       if path_arg
         logger.info("Setting the path in start_logger has no effect anymore, set it in the config instead")

data/lib/appsignal/config.rb

@@ -65,6 +65,7 @@ module Appsignal
       "APPSIGNAL_ENABLE_GC_INSTRUMENTATION"      => :enable_gc_instrumentation,
       "APPSIGNAL_RUNNING_IN_CONTAINER"           => :running_in_container,
       "APPSIGNAL_WORKING_DIR_PATH"               => :working_dir_path,
+      "APPSIGNAL_WORKING_DIRECTORY_PATH"         => :working_directory_path,
       "APPSIGNAL_ENABLE_HOST_METRICS"            => :enable_host_metrics,
       "APPSIGNAL_ENABLE_MINUTELY_PROBES"         => :enable_minutely_probes,
       "APPSIGNAL_HOSTNAME"                       => :hostname,
@@ -168,6 +169,7 @@ module Appsignal
       ENV["_APPSIGNAL_SEND_PARAMS"]                  = config_hash[:send_params].to_s
       ENV["_APPSIGNAL_RUNNING_IN_CONTAINER"]         = config_hash[:running_in_container].to_s
       ENV["_APPSIGNAL_WORKING_DIR_PATH"]             = config_hash[:working_dir_path] if config_hash[:working_dir_path]
+      ENV["_APPSIGNAL_WORKING_DIRECTORY_PATH"]       = config_hash[:working_directory_path] if config_hash[:working_directory_path]
       ENV["_APPSIGNAL_ENABLE_HOST_METRICS"]          = config_hash[:enable_host_metrics].to_s
       ENV["_APPSIGNAL_ENABLE_MINUTELY_PROBES"]       = config_hash[:enable_minutely_probes].to_s
       ENV["_APPSIGNAL_HOSTNAME"]                     = config_hash[:hostname].to_s
@@ -246,6 +248,12 @@ module Appsignal
           next if config[new_key] # Skip if new key is already in use
           config[new_key] = old_config_value
         end
+
+        if config.include?(:working_dir_path)
+          logger.warn "'working_dir_path' is deprecated, please use " \
+                      "'working_directory_path' instead and specify the " \
+                      "full path to the working directory"
+        end
       end
     end
 

data/lib/appsignal/extension/jruby.rb

@@ -72,7 +72,7 @@ module Appsignal
           [:appsignal_string, :double],
           :void
         attach_function :appsignal_increment_counter,
-          [:appsignal_string, :int64, :pointer],
+          [:appsignal_string, :double, :pointer],
           :void
         attach_function :appsignal_add_distribution_value,
           [:appsignal_string, :double, :pointer],

data/lib/appsignal/integrations/mongo_ruby_driver.rb

@@ -48,7 +48,7 @@ module Appsignal
         transaction.finish_event(
           "query.mongodb",
           "#{event.command_name} | #{event.database_name} | #{result}",
-          Appsignal::Utils.data_generate(command),
+          Appsignal::Utils::Data.generate(command),
           Appsignal::EventFormatter::DEFAULT
         )
       end

data/lib/appsignal/js_exception_transaction.rb

@@ -29,7 +29,7 @@ module Appsignal
       ext.set_error(
         @data["name"],
         @data["message"] || "",
-        Appsignal::Utils.data_generate(@data["backtrace"] || [])
+        Appsignal::Utils::Data.generate(@data["backtrace"] || [])
       )
     end
 
@@ -43,7 +43,7 @@ module Appsignal
         next unless data.is_a?(Array) || data.is_a?(Hash)
         ext.set_sample_data(
           key.to_s,
-          Appsignal::Utils.data_generate(data)
+          Appsignal::Utils::Data.generate(data)
         )
       end
     end

data/lib/appsignal/transaction.rb

@@ -9,6 +9,8 @@ module Appsignal
     ACTION_CABLE   = "action_cable".freeze
     FRONTEND       = "frontend".freeze
     BLANK          = "".freeze
+    ALLOWED_TAG_KEY_TYPES = [Symbol, String].freeze
+    ALLOWED_TAG_VALUE_TYPES = [Symbol, String, Integer].freeze
 
     class << self
       def create(id, namespace, request, options = {})
@@ -242,7 +244,7 @@ module Appsignal
       return unless key && data && (data.is_a?(Array) || data.is_a?(Hash))
       @ext.set_sample_data(
         key.to_s,
-        Appsignal::Utils.data_generate(data)
+        Appsignal::Utils::Data.generate(data)
       )
     rescue RuntimeError => e
       Appsignal.logger.error("Error generating data (#{e.class}: #{e.message}) for '#{data.inspect}'")
@@ -268,7 +270,7 @@ module Appsignal
       @ext.set_error(
         error.class.name,
         error.message.to_s,
-        backtrace ? Appsignal::Utils.data_generate(backtrace) : Appsignal::Extension.data_array_new
+        backtrace ? Appsignal::Utils::Data.generate(backtrace) : Appsignal::Extension.data_array_new
       )
     end
     alias_method :add_exception, :set_error
@@ -447,10 +449,12 @@ module Appsignal
     # * Key is a symbol or string with less then 100 chars
     # * Value is a symbol or string with less then 100 chars
     # * Value is an integer
+    #
+    # @see https://docs.appsignal.com/ruby/instrumentation/tagging.html
     def sanitized_tags
-      @tags.select do |k, v|
-        (k.is_a?(Symbol) || k.is_a?(String) && k.length <= 100) &&
-          (((v.is_a?(Symbol) || v.is_a?(String)) && v.length <= 100) || v.is_a?(Integer))
+      @tags.select do |key, value|
+        ALLOWED_TAG_KEY_TYPES.any? { |type| key.is_a? type } &&
+          ALLOWED_TAG_VALUE_TYPES.any? { |type| value.is_a? type }
       end
     end
 

data/lib/appsignal/utils.rb

@@ -1,129 +1,6 @@
 # frozen_string_literal: true
 
+require "appsignal/utils/data"
 require "appsignal/utils/hash_sanitizer"
+require "appsignal/utils/json"
 require "appsignal/utils/query_params_sanitizer"
-
-module Appsignal
-  # @api private
-  module Utils
-    def self.data_generate(body)
-      Data.generate(body)
-    end
-
-    class Data
-      class << self
-        def generate(body)
-          if body.is_a?(Hash)
-            map_hash(body)
-          elsif body.is_a?(Array)
-            map_array(body)
-          else
-            raise TypeError, "Body of type #{body.class} should be a Hash or Array"
-          end
-        end
-
-        def map_hash(hash_value)
-          map = Appsignal::Extension.data_map_new
-          hash_value.each do |key, value|
-            key = key.to_s
-            case value
-            when String
-              map.set_string(key, value)
-            when Integer
-              # An Integer too big for C-lang longs to fit
-              bigint = 1 << 63
-              if value >= bigint
-                map.set_string(key, "bigint:#{value}")
-              else
-                map.set_integer(key, value)
-              end
-            when Float
-              map.set_float(key, value)
-            when TrueClass, FalseClass
-              map.set_boolean(key, value)
-            when NilClass
-              map.set_nil(key)
-            when Hash
-              map.set_data(key, map_hash(value))
-            when Array
-              map.set_data(key, map_array(value))
-            else
-              map.set_string(key, value.to_s)
-            end
-          end
-          map
-        end
-
-        def map_array(array_value)
-          array = Appsignal::Extension.data_array_new
-          array_value.each do |value|
-            case value
-            when String
-              array.append_string(value)
-            when Integer
-              # An Integer too big for C-lang longs to fit
-              bigint = 1 << 63
-              if value >= bigint
-                array.append_string("bigint:#{value}")
-              else
-                array.append_integer(value)
-              end
-            when Float
-              array.append_float(value)
-            when TrueClass, FalseClass
-              array.append_boolean(value)
-            when NilClass
-              array.append_nil
-            when Hash
-              array.append_data(map_hash(value))
-            when Array
-              array.append_data(map_array(value))
-            else
-              array.append_string(value.to_s)
-            end
-          end
-          array
-        end
-      end
-    end
-
-    def self.json_generate(body)
-      JSON.generate(body)
-    end
-
-    class JSON
-      class << self
-        def generate(body)
-          ::JSON.generate(jsonify(body))
-        end
-
-        private
-
-        def jsonify(value)
-          case value
-          when String
-            encode_utf8(value)
-          when Numeric, NilClass, TrueClass, FalseClass
-            value
-          when Hash
-            value.each_with_object({}) do |(k, v), hash|
-              hash[jsonify(k)] = jsonify(v)
-            end
-          when Array
-            value.map { |v| jsonify(v) }
-          else
-            jsonify(value.to_s)
-          end
-        end
-
-        def encode_utf8(value)
-          value.encode(
-            "utf-8".freeze,
-            :invalid => :replace,
-            :undef   => :replace
-          )
-        end
-      end
-    end
-  end
-end

data/lib/appsignal/utils/data.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+module Appsignal
+  module Utils
+    # @api private
+    class Data
+      class << self
+        def generate(body)
+          if body.is_a?(Hash)
+            map_hash(body)
+          elsif body.is_a?(Array)
+            map_array(body)
+          else
+            raise TypeError, "Body of type #{body.class} should be a Hash or Array"
+          end
+        end
+
+        def map_hash(hash_value)
+          map = Appsignal::Extension.data_map_new
+          hash_value.each do |key, value|
+            key = key.to_s
+            case value
+            when String
+              map.set_string(key, value)
+            when Integer
+              # An Integer too big for C-lang longs to fit
+              bigint = 1 << 63
+              if value >= bigint
+                map.set_string(key, "bigint:#{value}")
+              else
+                map.set_integer(key, value)
+              end
+            when Float
+              map.set_float(key, value)
+            when TrueClass, FalseClass
+              map.set_boolean(key, value)
+            when NilClass
+              map.set_nil(key)
+            when Hash
+              map.set_data(key, map_hash(value))
+            when Array
+              map.set_data(key, map_array(value))
+            else
+              map.set_string(key, value.to_s)
+            end
+          end
+          map
+        end
+
+        def map_array(array_value)
+          array = Appsignal::Extension.data_array_new
+          array_value.each do |value|
+            case value
+            when String
+              array.append_string(value)
+            when Integer
+              # An Integer too big for C-lang longs to fit
+              bigint = 1 << 63
+              if value >= bigint
+                array.append_string("bigint:#{value}")
+              else
+                array.append_integer(value)
+              end
+            when Float
+              array.append_float(value)
+            when TrueClass, FalseClass
+              array.append_boolean(value)
+            when NilClass
+              array.append_nil
+            when Hash
+              array.append_data(map_hash(value))
+            when Array
+              array.append_data(map_array(value))
+            else
+              array.append_string(value.to_s)
+            end
+          end
+          array
+        end
+      end
+    end
+  end
+end

data/lib/appsignal/utils/json.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module Appsignal
+  module Utils
+    # @api private
+    class JSON
+      class << self
+        def generate(body)
+          ::JSON.generate(jsonify(body))
+        end
+
+        private
+
+        def jsonify(value)
+          case value
+          when String
+            encode_utf8(value)
+          when Numeric, NilClass, TrueClass, FalseClass
+            value
+          when Hash
+            value.each_with_object({}) do |(k, v), hash|
+              hash[jsonify(k)] = jsonify(v)
+            end
+          when Array
+            value.map { |v| jsonify(v) }
+          else
+            jsonify(value.to_s)
+          end
+        end
+
+        def encode_utf8(value)
+          value.encode(
+            "utf-8".freeze,
+            :invalid => :replace,
+            :undef   => :replace
+          )
+        end
+      end
+    end
+  end
+end

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Appsignal
-  VERSION = "2.7.0.alpha.4".freeze
+  VERSION = "2.7.0.beta.1".freeze
 end

data/spec/lib/appsignal/config_spec.rb

@@ -443,6 +443,7 @@ describe Appsignal::Config do
       expect(ENV["_APPSIGNAL_FILES_WORLD_ACCESSIBLE"]).to       eq "true"
       expect(ENV["_APP_REVISION"]).to                           eq "v2.5.1"
       expect(ENV).to_not                                        have_key("_APPSIGNAL_WORKING_DIR_PATH")
+      expect(ENV).to_not                                        have_key("_APPSIGNAL_WORKING_DIRECTORY_PATH")
     end
 
     context "with :hostname" do
@@ -466,6 +467,17 @@ describe Appsignal::Config do
         expect(ENV["_APPSIGNAL_WORKING_DIR_PATH"]).to eq "/tmp/appsignal2"
       end
     end
+
+    context "with :working_directory_path" do
+      before do
+        config[:working_directory_path] = "/tmp/appsignal2"
+        config.write_to_environment
+      end
+
+      it "sets the modified :working_directory_path" do
+        expect(ENV["_APPSIGNAL_WORKING_DIRECTORY_PATH"]).to eq "/tmp/appsignal2"
+      end
+    end
   end
 
   describe "#log_file_path" do

data/spec/lib/appsignal/extension_spec.rb

@@ -107,7 +107,7 @@ describe Appsignal::Extension do
       end
 
       it "should have a increment_counter method" do
-        subject.increment_counter("key", 1, Appsignal::Extension.data_map_new)
+        subject.increment_counter("key", 1.0, Appsignal::Extension.data_map_new)
       end
 
       it "should have a add_distribution_value method" do

data/spec/lib/appsignal/integrations/mongo_ruby_driver_spec.rb

@@ -81,7 +81,7 @@ describe Appsignal::Hooks::MongoMonitorSubscriber do
         expect(transaction).to receive(:finish_event).with(
           "query.mongodb",
           "find | test | SUCCEEDED",
-          Appsignal::Utils.data_generate("foo" => "?"),
+          Appsignal::Utils::Data.generate("foo" => "?"),
           0
         )
 

data/spec/lib/appsignal/js_exception_transaction_spec.rb

@@ -60,7 +60,7 @@ describe Appsignal::JSExceptionTransaction do
       expect(transaction.ext).to receive(:set_error).with(
         "TypeError",
         "foo is not a valid method",
-        Appsignal::Utils.data_generate(["foo.bar/js:11:1", "foo.bar/js:22:2"])
+        Appsignal::Utils::Data.generate(["foo.bar/js:11:1", "foo.bar/js:22:2"])
       )
 
       transaction.set_error
@@ -71,7 +71,7 @@ describe Appsignal::JSExceptionTransaction do
     it "should call `Appsignal::Extension.set_transaction_error_data`" do
       expect(transaction.ext).to receive(:set_sample_data).with(
         "tags",
-        Appsignal::Utils.data_generate(["tag1"])
+        Appsignal::Utils::Data.generate(["tag1"])
       )
 
       transaction.set_sample_data
@@ -102,7 +102,7 @@ describe Appsignal::JSExceptionTransaction do
         expect(transaction.ext).to receive(:set_error).with(
           "TypeError",
           "",
-          Appsignal::Utils.data_generate([])
+          Appsignal::Utils::Data.generate([])
         )
 
         transaction.set_error

data/spec/lib/appsignal/transaction_spec.rb

@@ -334,68 +334,114 @@ describe Appsignal::Transaction do
     end
 
     describe "#set_tags" do
-      it "should add tags to transaction" do
-        expect do
-          transaction.set_tags("a" => "b")
-        end.to change(transaction, :tags).to("a" => "b")
+      let(:long_string) { "a" * 2001 }
+      before do
+        transaction.set_tags(
+          :valid_key => "valid_value",
+          "valid_string_key" => "valid_value",
+          :both_symbols => :valid_value,
+          :integer_value => 1,
+          :hash_value => { "invalid" => "hash" },
+          :array_value => %w[invalid array],
+          :object => Object.new,
+          :too_long_value => long_string,
+          long_string => "too_long_key"
+        )
+        transaction.sample_data
       end
-    end
 
-    describe "set_action" do
-      it "should set the action in extension" do
-        expect(transaction.ext).to receive(:set_action).with(
-          "PagesController#show"
-        ).once
+      it "stores tags on the transaction" do
+        expect(transaction.to_h["sample_data"]["tags"]).to eq(
+          "valid_key" => "valid_value",
+          "valid_string_key" => "valid_value",
+          "both_symbols" => "valid_value",
+          "integer_value" => 1,
+          "too_long_value" => "#{"a" * 2000}...",
+          long_string => "too_long_key"
+        )
+      end
+    end
 
-        transaction.set_action("PagesController#show")
+    describe "#set_action" do
+      context "when the action is set" do
+        it "updates the action name on the transaction" do
+          action_name = "PagesController#show"
+          transaction.set_action(action_name)
 
-        expect(transaction.action).to eq "PagesController#show"
+          expect(transaction.action).to eq(action_name)
+          expect(transaction.to_h["action"]).to eq(action_name)
+        end
       end
 
-      it "should not set the action in extension when value is nil" do
-        expect(Appsignal::Extension).to_not receive(:set_action)
+      context "when the action is nil" do
+        it "does not update the action name on the transaction" do
+          action_name = "PagesController#show"
+          transaction.set_action(action_name)
+          transaction.set_action(nil)
 
-        transaction.set_action(nil)
+          expect(transaction.action).to eq(action_name)
+          expect(transaction.to_h["action"]).to eq(action_name)
+        end
       end
     end
 
-    describe "set_action_if_nil" do
-      context "if action is currently nil" do
-        it "should set the action" do
-          expect(transaction.ext).to receive(:set_action).with(
-            "PagesController#show"
-          ).once
+    describe "#set_action_if_nil" do
+      context "when the action is not set" do
+        it "updates the action name on the transaction" do
+          expect(transaction.action).to eq(nil)
+          expect(transaction.to_h["action"]).to eq(nil)
+
+          action_name = "PagesController#show"
+          transaction.set_action_if_nil(action_name)
+
+          expect(transaction.action).to eq(action_name)
+          expect(transaction.to_h["action"]).to eq(action_name)
+        end
+
+        context "when the given action is nil" do
+          it "does not update the action name on the transaction" do
+            action_name = "something"
+            transaction.set_action("something")
+            transaction.set_action_if_nil(nil)
 
-          transaction.set_action_if_nil("PagesController#show")
+            expect(transaction.action).to eq(action_name)
+            expect(transaction.to_h["action"]).to eq(action_name)
+          end
         end
       end
 
-      context "if action is currently set" do
-        it "should not set the action" do
+      context "when the action is set" do
+        it "does not update the action name on the transaction" do
+          action_name = "something"
           transaction.set_action("something")
+          transaction.set_action_if_nil("something else")
 
-          expect(transaction.ext).not_to receive(:set_action)
-
-          transaction.set_action_if_nil("PagesController#show")
+          expect(transaction.action).to eq(action_name)
+          expect(transaction.to_h["action"]).to eq(action_name)
         end
       end
     end
 
-    describe "set_namespace" do
-      it "should set the action in extension" do
-        expect(transaction.ext).to receive(:set_namespace).with(
-          "custom"
-        ).once
+    describe "#set_namespace" do
+      context "when the namespace is not nil" do
+        it "updates the namespace on the transaction" do
+          namespace = "custom"
+          transaction.set_namespace(namespace)
 
-        transaction.set_namespace("custom")
-
-        expect(transaction.namespace).to eq "custom"
+          expect(transaction.namespace).to eq namespace
+          expect(transaction.to_h["namespace"]).to eq(namespace)
+        end
       end
 
-      it "should not set the action in extension when value is nil" do
-        expect(Appsignal::Extension).to_not receive(:set_namespace)
+      context "when the namespace is nil" do
+        it "does not update the namespace on the transaction" do
+          namespace = "custom"
+          transaction.set_namespace(namespace)
+          transaction.set_namespace(nil)
 
-        transaction.set_action(nil)
+          expect(transaction.namespace).to eq(namespace)
+          expect(transaction.to_h["namespace"]).to eq(namespace)
+        end
       end
     end
 
@@ -478,44 +524,68 @@ describe Appsignal::Transaction do
     end
 
     describe "#set_metadata" do
-      it "should set the metdata in extension" do
-        expect(transaction.ext).to receive(:set_metadata).with(
-          "request_method",
-          "GET"
-        ).once
-
+      it "updates the metadata on the transaction" do
         transaction.set_metadata("request_method", "GET")
+
+        expect(transaction.to_h["metadata"]).to eq("request_method" => "GET")
       end
 
-      it "should not set the metdata in extension when value is nil" do
-        expect(transaction.ext).to_not receive(:set_metadata)
+      context "when the key is nil" do
+        it "does not update the metadata on the transaction" do
+          transaction.set_metadata(nil, "GET")
 
-        transaction.set_metadata("request_method", nil)
+          expect(transaction.to_h["metadata"]).to eq({})
+        end
       end
-    end
 
-    describe "set_sample_data" do
-      it "should set the data" do
-        expect(transaction.ext).to receive(:set_sample_data).with(
-          "params",
-          Appsignal::Utils.data_generate("controller" => "blog_posts", "action" => "show", "id" => "1")
-        ).once
+      context "when the value is nil" do
+        it "does not update the metadata on the transaction" do
+          transaction.set_metadata("request_method", nil)
 
+          expect(transaction.to_h["metadata"]).to eq({})
+        end
+      end
+    end
+
+    describe "#set_sample_data" do
+      it "updates the sample data on the transaction" do
         transaction.set_sample_data(
           "params",
           :controller => "blog_posts",
           :action     => "show",
           :id         => "1"
         )
-      end
 
-      it "should do nothing if the data cannot be converted to json" do
-        expect(transaction.ext).to_not receive(:set_sample_data).with(
-          "params",
-          kind_of(String)
+        expect(transaction.to_h["sample_data"]).to eq(
+          "params" => {
+            "action" => "show",
+            "controller" => "blog_posts",
+            "id" => "1"
+          }
         )
+      end
+
+      context "when the data is no Array or Hash" do
+        it "does not update the sample data on the transaction" do
+          transaction.set_sample_data("params", "string")
+
+          expect(transaction.to_h["sample_data"]).to eq({})
+        end
+      end
 
-        transaction.set_sample_data("params", "string")
+      context "when the data cannot be converted to JSON" do
+        it "does not update the sample data on the transaction" do
+          klass = Class.new do
+            def to_s
+              raise "foo" # Cause a deliberate error
+            end
+          end
+          transaction.set_sample_data("params", klass.new => 1)
+
+          expect(transaction.to_h["sample_data"]).to eq({})
+          expect(log_contents(log)).to contains_log :error,
+            "Error generating data (RuntimeError: foo) for"
+        end
       end
     end
 
@@ -523,7 +593,7 @@ describe Appsignal::Transaction do
       it "should sample data" do
         expect(transaction.ext).to receive(:set_sample_data).with(
           "environment",
-          Appsignal::Utils.data_generate(
+          Appsignal::Utils::Data.generate(
             "CONTENT_LENGTH" => "0",
             "REQUEST_METHOD" => "GET",
             "SERVER_NAME" => "example.org",
@@ -533,19 +603,23 @@ describe Appsignal::Transaction do
         ).once
         expect(transaction.ext).to receive(:set_sample_data).with(
           "session_data",
-          Appsignal::Utils.data_generate({})
+          Appsignal::Utils::Data.generate({})
         ).once
         expect(transaction.ext).to receive(:set_sample_data).with(
           "params",
-          Appsignal::Utils.data_generate("controller" => "blog_posts", "action" => "show", "id" => "1")
+          Appsignal::Utils::Data.generate(
+            "controller" => "blog_posts",
+            "action" => "show",
+            "id" => "1"
+          )
         ).once
         expect(transaction.ext).to receive(:set_sample_data).with(
           "metadata",
-          Appsignal::Utils.data_generate("key" => "value")
+          Appsignal::Utils::Data.generate("key" => "value")
         ).once
         expect(transaction.ext).to receive(:set_sample_data).with(
           "tags",
-          Appsignal::Utils.data_generate({})
+          Appsignal::Utils::Data.generate({})
         ).once
 
         transaction.sample_data
@@ -572,7 +646,7 @@ describe Appsignal::Transaction do
           expect(transaction.ext).to receive(:set_error).with(
             "RSpec::Mocks::Double",
             "test message",
-            Appsignal::Utils.data_generate(["line 1"])
+            Appsignal::Utils::Data.generate(["line 1"])
           )
 
           transaction.set_error(error)
@@ -590,7 +664,7 @@ describe Appsignal::Transaction do
           expect(transaction.ext).to receive(:set_error).with(
             "RSpec::Mocks::Double",
             "",
-            Appsignal::Utils.data_generate(["line 1"])
+            Appsignal::Utils::Data.generate(["line 1"])
           )
 
           transaction.set_error(error)
@@ -1080,32 +1154,6 @@ describe Appsignal::Transaction do
       end
     end
 
-    describe "#sanitized_tags" do
-      before do
-        transaction.set_tags(
-          :valid_key => "valid_value",
-          "valid_string_key" => "valid_value",
-          :both_symbols => :valid_value,
-          :integer_value => 1,
-          :hash_value => { "invalid" => "hash" },
-          :array_value => %w[invalid array],
-          :to_long_value => SecureRandom.urlsafe_base64(101),
-          :object => Object.new,
-          SecureRandom.urlsafe_base64(101) => "to_long_key"
-        )
-      end
-      subject { transaction.send(:sanitized_tags).keys }
-
-      it "should only return whitelisted data" do
-        is_expected.to match_array([
-          :valid_key,
-          "valid_string_key",
-          :both_symbols,
-          :integer_value
-        ])
-      end
-    end
-
     describe "#cleaned_backtrace" do
       subject { transaction.send(:cleaned_backtrace, ["line 1", "line 2"]) }
 

data/spec/lib/appsignal/{utils_spec.rb → utils/data_spec.rb}

@@ -1,8 +1,8 @@
 # encoding: UTF-8
 
-describe Appsignal::Utils do
-  describe ".data_generate" do
-    subject { Appsignal::Utils.data_generate(body) }
+describe Appsignal::Utils::Data do
+  describe ".generate" do
+    subject { Appsignal::Utils::Data.generate(body) }
 
     context "with a valid hash body" do
       let(:body) do
@@ -22,8 +22,8 @@ describe Appsignal::Utils do
         }
       end
 
-      it { is_expected.to eq Appsignal::Utils.data_generate(body) }
-      it { is_expected.to_not eq Appsignal::Utils.data_generate({}) }
+      it { is_expected.to eq Appsignal::Utils::Data.generate(body) }
+      it { is_expected.to_not eq Appsignal::Utils::Data.generate({}) }
 
       describe "#to_s" do
         it "returns a serialized hash" do
@@ -60,8 +60,8 @@ describe Appsignal::Utils do
         ]
       end
 
-      it { is_expected.to eq Appsignal::Utils.data_generate(body) }
-      it { is_expected.to_not eq Appsignal::Utils.data_generate({}) }
+      it { is_expected.to eq Appsignal::Utils::Data.generate(body) }
+      it { is_expected.to_not eq Appsignal::Utils::Data.generate({}) }
 
       describe "#to_s" do
         it "returns a serialized array" do
@@ -110,41 +110,4 @@ describe Appsignal::Utils do
       end
     end
   end
-
-  describe ".json_generate" do
-    subject { Appsignal::Utils.json_generate(body) }
-
-    context "with a valid body" do
-      let(:body) do
-        {
-          "the" => "payload",
-          1 => true,
-          nil => "test",
-          :foo => [1, 2, "three"],
-          "bar" => nil,
-          "baz" => { "foo" => "bar" }
-        }
-      end
-
-      it { is_expected.to eq %({"the":"payload","1":true,"":"test","foo":[1,2,"three"],"bar":null,"baz":{"foo":"bar"}}) }
-    end
-
-    context "with a body that contains strings with invalid utf-8 content" do
-      let(:string_with_invalid_utf8) { [0x61, 0x61, 0x85].pack("c*") }
-      let(:body) do
-        {
-          "field_one" => [0x61, 0x61].pack("c*"),
-          :field_two => string_with_invalid_utf8,
-          "field_three" => [
-            "one", string_with_invalid_utf8
-          ],
-          "field_four" => {
-            "one" => string_with_invalid_utf8
-          }
-        }
-      end
-
-      it { is_expected.to eq %({"field_one":"aa","field_two":"aa�","field_three":["one","aa�"],"field_four":{"one":"aa�"}}) }
-    end
-  end
 end

data/spec/lib/appsignal/utils/json_spec.rb

@@ -0,0 +1,40 @@
+# encoding: UTF-8
+
+describe Appsignal::Utils::JSON do
+  describe ".generate" do
+    subject { Appsignal::Utils::JSON.generate(body) }
+
+    context "with a valid body" do
+      let(:body) do
+        {
+          "the" => "payload",
+          1 => true,
+          nil => "test",
+          :foo => [1, 2, "three"],
+          "bar" => nil,
+          "baz" => { "foo" => "bar" }
+        }
+      end
+
+      it { is_expected.to eq %({"the":"payload","1":true,"":"test","foo":[1,2,"three"],"bar":null,"baz":{"foo":"bar"}}) }
+    end
+
+    context "with a body that contains strings with invalid utf-8 content" do
+      let(:string_with_invalid_utf8) { [0x61, 0x61, 0x85].pack("c*") }
+      let(:body) do
+        {
+          "field_one" => [0x61, 0x61].pack("c*"),
+          :field_two => string_with_invalid_utf8,
+          "field_three" => [
+            "one", string_with_invalid_utf8
+          ],
+          "field_four" => {
+            "one" => string_with_invalid_utf8
+          }
+        }
+      end
+
+      it { is_expected.to eq %({"field_one":"aa","field_two":"aa�","field_three":["one","aa�"],"field_four":{"one":"aa�"}}) }
+    end
+  end
+end

data/spec/lib/appsignal_spec.rb

@@ -434,7 +434,8 @@ describe Appsignal do
         end
 
         it "should call set_gauge with tags" do
-          expect(Appsignal::Extension).to receive(:set_gauge).with("key", 0.1, Appsignal::Utils.data_generate(tags))
+          expect(Appsignal::Extension).to receive(:set_gauge)
+            .with("key", 0.1, Appsignal::Utils::Data.generate(tags))
           Appsignal.set_gauge("key", 0.1, tags)
         end
 
@@ -499,7 +500,8 @@ describe Appsignal do
         end
 
         it "should call increment_counter with tags" do
-          expect(Appsignal::Extension).to receive(:increment_counter).with("key", 1, Appsignal::Utils.data_generate(tags))
+          expect(Appsignal::Extension).to receive(:increment_counter)
+            .with("key", 1, Appsignal::Utils::Data.generate(tags))
           Appsignal.increment_counter("key", 1, tags)
         end
 
@@ -529,7 +531,8 @@ describe Appsignal do
         end
 
         it "should call add_distribution_value with tags" do
-          expect(Appsignal::Extension).to receive(:add_distribution_value).with("key", 0.1, Appsignal::Utils.data_generate(tags))
+          expect(Appsignal::Extension).to receive(:add_distribution_value)
+            .with("key", 0.1, Appsignal::Utils::Data.generate(tags))
           Appsignal.add_distribution_value("key", 0.1, tags)
         end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 2.7.0.alpha.4
+  version: 2.7.0.beta.1
 platform: ruby
 authors:
 - Robert Beekman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-29 00:00:00.000000000 Z
+date: 2018-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -235,7 +235,9 @@ files:
 - lib/appsignal/transaction.rb
 - lib/appsignal/transmitter.rb
 - lib/appsignal/utils.rb
+- lib/appsignal/utils/data.rb
 - lib/appsignal/utils/hash_sanitizer.rb
+- lib/appsignal/utils/json.rb
 - lib/appsignal/utils/query_params_sanitizer.rb
 - lib/appsignal/version.rb
 - lib/sequel/extensions/appsignal_integration.rb
@@ -304,9 +306,10 @@ files:
 - spec/lib/appsignal/system_spec.rb
 - spec/lib/appsignal/transaction_spec.rb
 - spec/lib/appsignal/transmitter_spec.rb
+- spec/lib/appsignal/utils/data_spec.rb
 - spec/lib/appsignal/utils/hash_sanitizer_spec.rb
+- spec/lib/appsignal/utils/json_spec.rb
 - spec/lib/appsignal/utils/query_params_sanitizer_spec.rb
-- spec/lib/appsignal/utils_spec.rb
 - spec/lib/appsignal_spec.rb
 - spec/spec_helper.rb
 - spec/support/fixtures/containers/cgroups/docker
@@ -362,7 +365,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.2.3
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com
@@ -430,9 +433,10 @@ test_files:
 - spec/lib/appsignal/system_spec.rb
 - spec/lib/appsignal/transaction_spec.rb
 - spec/lib/appsignal/transmitter_spec.rb
+- spec/lib/appsignal/utils/data_spec.rb
 - spec/lib/appsignal/utils/hash_sanitizer_spec.rb
+- spec/lib/appsignal/utils/json_spec.rb
 - spec/lib/appsignal/utils/query_params_sanitizer_spec.rb
-- spec/lib/appsignal/utils_spec.rb
 - spec/lib/appsignal_spec.rb
 - spec/spec_helper.rb
 - spec/support/fixtures/containers/cgroups/docker