appsignal 2.6.0 → 2.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8dc71a44278aeb3ebfa9386916e449696bc318de
-  data.tar.gz: a335e072cff746c820793c24915e6db621f07cdd
+  metadata.gz: 94769b5fa8923878f0bf0b9aa6cee8eb4369b59a
+  data.tar.gz: 73b22d4abec0b901146828b0260fe2b78af87fec
 SHA512:
-  metadata.gz: 7905745f4510f334eeb36ec61b15e53b000a658ab1b5d47054743e4629ec99541652bce7df9b4beadaaf3e4ce8ff235102a9010f981e3f495cfc6d16c787b052
-  data.tar.gz: 129eeb7aeee1d1728cc21ba69cb894d8901b1d6d5c067e695c3c6bd6eecd791469e74831fada0cee7811b6a6323cd70badf6472870b26a44ef9b561c03d0eebc
+  metadata.gz: 55e2d6076ab03a05c1f99a675cf295da6f58e1f79d59e7c059f541a365c6e99274ee168d528ac3196f451cebd608c462f0b00c324ef7906aac6caa928ea66467
+  data.tar.gz: 5a449893cff0d75f63cfc9969f4e7f2720f1031d1ad431c478748a4a9b549389052373a525654184b51337fca0559fcc5efa1d431df84586ae6524850653df71

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 2.6.1
+- Remove request_headers warning and use sane default. PR #410
+- Fix metrics format for internal agent metrics. PR #411
+
 # 2.6.0
 - Enable frozen strings by default. PR #384
 - Add `revision` config option. PR #388

data/ext/agent.yml

@@ -1,64 +1,64 @@
 ---
-version: 6ce11c8
+version: e509c90
 triples:
   x86_64-darwin:
     static:
-      checksum: 270f44902bcd278277b4e5de598fd081145a35f09ccdad1bc2511cd83d37c2f7
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-darwin-all-static.tar.gz
+      checksum: f80f79f80afee3bdfb5f09f58503e6912fe14b74311baf07ac4f888f645e1b0d
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-darwin-all-static.tar.gz
     dynamic:
-      checksum: 502b5ac7f83c70c27f768181dc4e2a4e6d27a794c0221a0d0c9977709ba2e98b
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-darwin-all-dynamic.tar.gz
+      checksum: 934b6dbc77e706475dc564ec1fee87d6b97ada27233cfb4a92abaa42cc8f06ed
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-darwin-all-dynamic.tar.gz
   universal-darwin:
     static:
-      checksum: 270f44902bcd278277b4e5de598fd081145a35f09ccdad1bc2511cd83d37c2f7
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-darwin-all-static.tar.gz
+      checksum: f80f79f80afee3bdfb5f09f58503e6912fe14b74311baf07ac4f888f645e1b0d
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-darwin-all-static.tar.gz
     dynamic:
-      checksum: 502b5ac7f83c70c27f768181dc4e2a4e6d27a794c0221a0d0c9977709ba2e98b
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-darwin-all-dynamic.tar.gz
+      checksum: 934b6dbc77e706475dc564ec1fee87d6b97ada27233cfb4a92abaa42cc8f06ed
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-darwin-all-dynamic.tar.gz
   i686-linux:
     static:
-      checksum: 3b7d7b01650d419a5a4cf08cbf149e60a9fedef2a21f06f48672c86fc6087bcc
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-i686-linux-all-static.tar.gz
+      checksum: 8f93646fe4d9769d7f132d7bd2fae4ef48cd12daff341b58e142ac4a29ff3cd0
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-i686-linux-all-static.tar.gz
     dynamic:
-      checksum: 2ff19f08d8f3d8aa3d78745d58cde3e8fe640ef28556bb885cbab1a9419f8cde
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-i686-linux-all-dynamic.tar.gz
+      checksum: 4e5ef5e2432ec5c0fc895d1cf97ee2d0e990e45c63369c2cd0c6bf4dbed621f8
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-i686-linux-all-dynamic.tar.gz
   x86-linux:
     static:
-      checksum: 3b7d7b01650d419a5a4cf08cbf149e60a9fedef2a21f06f48672c86fc6087bcc
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-i686-linux-all-static.tar.gz
+      checksum: 8f93646fe4d9769d7f132d7bd2fae4ef48cd12daff341b58e142ac4a29ff3cd0
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-i686-linux-all-static.tar.gz
     dynamic:
-      checksum: 2ff19f08d8f3d8aa3d78745d58cde3e8fe640ef28556bb885cbab1a9419f8cde
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-i686-linux-all-dynamic.tar.gz
+      checksum: 4e5ef5e2432ec5c0fc895d1cf97ee2d0e990e45c63369c2cd0c6bf4dbed621f8
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-i686-linux-all-dynamic.tar.gz
   i686-linux-musl:
     static:
-      checksum: c5a0b5f156e94ab9ab763ebd031d002d53b2905d8d5d44eecaecea4f3b757cb5
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-i686-linux-musl-all-static.tar.gz
+      checksum: 460de28c80c8e892ed7b21e9e2c8fe14027a4de36145735c09df393904aac90b
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-i686-linux-musl-all-static.tar.gz
   x86-linux-musl:
     static:
-      checksum: c5a0b5f156e94ab9ab763ebd031d002d53b2905d8d5d44eecaecea4f3b757cb5
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-i686-linux-musl-all-static.tar.gz
+      checksum: 460de28c80c8e892ed7b21e9e2c8fe14027a4de36145735c09df393904aac90b
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-i686-linux-musl-all-static.tar.gz
   x86_64-linux:
     static:
-      checksum: 2c3f49329e6549a0f168238ed771d89f128ce3ac56a3d7f950ba58151fc2e03d
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-linux-all-static.tar.gz
+      checksum: 0de52c7fee6e651441ff11384e25e8d5c72065e7f99997ff32babd15d7ca2317
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-linux-all-static.tar.gz
     dynamic:
-      checksum: 2a6d28da207f52ff73b9fb9e9337621b95a7b2329eeb4f9da7a83156af519e20
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-linux-all-dynamic.tar.gz
+      checksum: 10874a13d8523de6d178b55e79ec3ff4a0c0f0e7a34c96d3731c382bcdb0ea1e
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-linux-all-dynamic.tar.gz
   x86_64-linux-musl:
     static:
-      checksum: 7a148a871f61f697eb921828ef93232574c71e8d0ea79d30cd4e27b4f5609bc2
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-linux-musl-all-static.tar.gz
+      checksum: c49e094d2a9894c98538e11aa7e519619e08daf30d7b7ae4d38e10bb44e14084
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-linux-musl-all-static.tar.gz
   x86_64-freebsd:
     static:
-      checksum: 5cc8e1702275930e4e1d213ad57b64d892d175b9e84242d757bd23b98ae35d85
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-freebsd-all-static.tar.gz
+      checksum: ad939a8919edfab55a442faebaf6095584a690e4b6a08a288ac7630b2069799c
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-freebsd-all-static.tar.gz
     dynamic:
-      checksum: 05d18c8571bb4baa5bafcfca10c8919f94713067cfe4b1ae3fc9e66ca561bb07
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-freebsd-all-dynamic.tar.gz
+      checksum: 3027b9ecb3e839b93ac5c692a2a5b8c1c2cda57c2aaf35f912156d17015a0485
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-freebsd-all-dynamic.tar.gz
   amd64-freebsd:
     static:
-      checksum: 5cc8e1702275930e4e1d213ad57b64d892d175b9e84242d757bd23b98ae35d85
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-freebsd-all-static.tar.gz
+      checksum: ad939a8919edfab55a442faebaf6095584a690e4b6a08a288ac7630b2069799c
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-freebsd-all-static.tar.gz
     dynamic:
-      checksum: 05d18c8571bb4baa5bafcfca10c8919f94713067cfe4b1ae3fc9e66ca561bb07
-      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/6ce11c8/appsignal-x86_64-freebsd-all-dynamic.tar.gz
+      checksum: 3027b9ecb3e839b93ac5c692a2a5b8c1c2cda57c2aaf35f912156d17015a0485
+      download_url: https://appsignal-agent-releases.global.ssl.fastly.net/e509c90/appsignal-x86_64-freebsd-all-dynamic.tar.gz

data/lib/appsignal/cli/install.rb

@@ -164,7 +164,7 @@ module Appsignal
           puts "How do you want to configure AppSignal?"
           puts "  (1) a config file"
           puts "  (2) environment variables"
-          loop do # rubocop:disable Metrics/BlockLength
+          loop do
             print "  Choose (1/2): "
             case ask_for_input
             when "1"
@@ -176,7 +176,6 @@ module Appsignal
               write_config_file(
                 :push_api_key => config[:push_api_key],
                 :app_name => config[:name],
-                :request_headers => multiline_request_headers,
                 :environments => environments
               )
               puts
@@ -192,7 +191,6 @@ module Appsignal
               if name_overwritten
                 puts "  export APPSIGNAL_APP_NAME=#{config[:name]}"
               end
-              puts "  export APPSIGNAL_REQUEST_HEADERS=#{single_line_request_headers}"
               puts
               puts "  See the documentation for more configuration options:"
               puts "  http://docs.appsignal.com/gem-settings/configuration.html"
@@ -275,16 +273,6 @@ module Appsignal
         def new_config
           Appsignal::Config.new(Dir.pwd, "")
         end
-
-        def multiline_request_headers
-          Appsignal::Config::SUGGESTED_REQUEST_HEADERS.map do |row|
-            row.map(&:inspect).join(", ")
-          end.join(",\n    ")
-        end
-
-        def single_line_request_headers
-          Appsignal::Config::SUGGESTED_REQUEST_HEADERS.flatten.join(",")
-        end
       end
     end
   end

data/lib/appsignal/config.rb

@@ -17,6 +17,13 @@ module Appsignal
       :filter_parameters              => [],
       :filter_session_data            => [],
       :send_params                    => true,
+      :request_headers                => %w[
+        HTTP_ACCEPT HTTP_ACCEPT_CHARSET HTTP_ACCEPT_ENCODING
+        HTTP_ACCEPT_LANGUAGE HTTP_CACHE_CONTROL HTTP_CONNECTION
+        CONTENT_LENGTH PATH_INFO HTTP_RANGE
+        REQUEST_METHOD REQUEST_URI SERVER_NAME SERVER_PORT
+        SERVER_PROTOCOL
+      ],
       :endpoint                       => "https://push.appsignal.com",
       :instrument_net_http            => true,
       :instrument_redis               => true,
@@ -67,14 +74,6 @@ module Appsignal
       "APPSIGNAL_REQUEST_HEADERS"                => :request_headers,
       "APP_REVISION"                             => :revision
     }.freeze
-    # Formatted in two-dimensional array for easy printing
-    SUGGESTED_REQUEST_HEADERS = [
-      %w[HTTP_ACCEPT HTTP_ACCEPT_CHARSET HTTP_ACCEPT_ENCODING],
-      %w[HTTP_ACCEPT_LANGUAGE HTTP_CACHE_CONTROL HTTP_CONNECTION],
-      %w[CONTENT_LENGTH PATH_INFO HTTP_RANGE],
-      %w[REQUEST_METHOD REQUEST_URI SERVER_NAME SERVER_PORT],
-      %w[SERVER_PROTOCOL]
-    ].freeze
 
     # Mapping of old and deprecated AppSignal configuration keys
     DEPRECATED_CONFIG_KEY_MAPPING = {
@@ -197,35 +196,10 @@ module Appsignal
         @valid = false
         @logger.error "Push api key not set after loading config"
       end
-
-      check_if_request_headers_option_is_set
     end
 
     private
 
-    def check_if_request_headers_option_is_set
-      return unless valid?
-      return if config_hash.key?(:request_headers) || env == "test"
-
-      multi_line_request_headers = SUGGESTED_REQUEST_HEADERS.map do |row|
-        row.map(&:inspect).join(", ")
-      end.join(",\n    ")
-      single_line_request_headers = SUGGESTED_REQUEST_HEADERS.flatten.join(",")
-
-      puts "Warning: The `request_headers` config option was not set " \
-        " in the AppSignal configuration, falling back to the default list. " \
-        "Please explicitly list response headers to send to AppSignal in " \
-        "config/appsignal.yml:\n\n" \
-        "  request_headers: [\n" \
-        "    #{multi_line_request_headers}\n" \
-        "]\n\n" \
-        "Or set the APPSIGNAL_REQUEST_HEADERS environment variable:\n\n"\
-        "  $ export APPSIGNAL_REQUEST_HEADERS=" \
-        "\"#{single_line_request_headers}\"\n\n" \
-        "Please check https://github.com/appsignal/appsignal-ruby/pull/406 " \
-        "for more information on this change."
-    end
-
     def config_file
       @config_file ||=
         root_path.nil? ? nil : File.join(root_path, "config", "appsignal.yml")

data/lib/appsignal/transaction.rb

@@ -10,21 +10,6 @@ module Appsignal
     FRONTEND       = "frontend".freeze
     BLANK          = "".freeze
 
-    # Based on what Rails uses + some variables we'd like to show
-    FALLBACK_REQUEST_HEADERS = %w[
-      CONTENT_LENGTH AUTH_TYPE GATEWAY_INTERFACE
-      PATH_TRANSLATED REMOTE_HOST REMOTE_IDENT REMOTE_USER REMOTE_ADDR
-      REQUEST_METHOD SERVER_NAME SERVER_PORT SERVER_PROTOCOL REQUEST_URI
-      PATH_INFO
-
-      HTTP_X_REQUEST_START HTTP_X_MIDDLEWARE_START HTTP_X_QUEUE_START
-      HTTP_X_QUEUE_TIME HTTP_X_HEROKU_QUEUE_WAIT_TIME HTTP_X_APPLICATION_START
-      HTTP_ACCEPT HTTP_ACCEPT_CHARSET HTTP_ACCEPT_ENCODING HTTP_ACCEPT_LANGUAGE
-      HTTP_CACHE_CONTROL HTTP_CONNECTION HTTP_USER_AGENT HTTP_FROM
-      HTTP_NEGOTIATE HTTP_PRAGMA HTTP_REFERER HTTP_X_FORWARDED_FOR
-      HTTP_CLIENT_IP HTTP_RANGE
-    ].freeze
-
     class << self
       def create(id, namespace, request, options = {})
         # Allow middleware to force a new transaction
@@ -405,9 +390,6 @@ module Appsignal
     # The environment of a transaction can contain a lot of information, not
     # all of it useful for debugging.
     #
-    # Only the values from the keys specified in {FALLBACK_REQUEST_HEADERS} are
-    # returned.
-    #
     # @return [nil] if no environment is present.
     # @return [Hash<String, Object>]
     def sanitized_environment
@@ -415,8 +397,7 @@ module Appsignal
       return if env.empty?
 
       {}.tap do |out|
-        (Appsignal.config[:request_headers] ||
-          FALLBACK_REQUEST_HEADERS).each do |key|
+        Appsignal.config[:request_headers].each do |key|
           out[key] = env[key] if env[key]
         end
       end

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Appsignal
-  VERSION = "2.6.0".freeze
+  VERSION = "2.6.1".freeze
 end

data/resources/appsignal.yml.erb

@@ -6,11 +6,6 @@ default: &defaults
   # Your app's name
   name: "<%= app_name %>"
 
-  # Request headers that should be recorded per sample
-  request_headers: [
-    <%= request_headers %>
-  ]
-
   # Actions that should not be monitored by AppSignal
   # ignore_actions:
   #   - ApplicationController#isup

data/spec/lib/appsignal/cli/install_spec.rb

@@ -39,16 +39,6 @@ describe Appsignal::CLI::Install do
       actual.include?("export APPSIGNAL_APP_NAME=#{name}")
     end
   end
-  define :include_env_request_headers do
-    match do |actual|
-      actual.include?(
-        "export APPSIGNAL_REQUEST_HEADERS=HTTP_ACCEPT,HTTP_ACCEPT_CHARSET," \
-        "HTTP_ACCEPT_ENCODING,HTTP_ACCEPT_LANGUAGE,HTTP_CACHE_CONTROL," \
-        "HTTP_CONNECTION,CONTENT_LENGTH,PATH_INFO,HTTP_RANGE," \
-        "REQUEST_METHOD,REQUEST_URI,SERVER_NAME,SERVER_PORT,SERVER_PROTOCOL"
-      )
-    end
-  end
 
   define :configure_app_name do |name|
     match do |file_contents|
@@ -60,17 +50,6 @@ describe Appsignal::CLI::Install do
       file_contents =~ /^  push_api_key: "#{key}"/
     end
   end
-  define :configure_request_headers do
-    match do |file_contents|
-      file_contents =~ /^  request_headers: \[/ &&
-        file_contents =~ /^    "HTTP_ACCEPT", "HTTP_ACCEPT_CHARSET", "HTTP_ACCEPT_ENCODING",/ &&
-        file_contents =~ /^    "HTTP_ACCEPT_LANGUAGE", "HTTP_CACHE_CONTROL", "HTTP_CONNECTION",/ &&
-        file_contents =~ /^    "CONTENT_LENGTH", "PATH_INFO", "HTTP_RANGE",/ &&
-        file_contents =~ /^    "REQUEST_METHOD", "REQUEST_URI", "SERVER_NAME", "SERVER_PORT",/ &&
-        file_contents =~ /^    "SERVER_PROTOCOL"/ &&
-        file_contents =~ /^  \]/
-    end
-  end
   define :configure_environment do |env|
     match do |file_contents|
       file_contents =~ /^#{env}:$/
@@ -293,7 +272,6 @@ describe Appsignal::CLI::Install do
           expect(output).to include_file_config
           expect(config_file).to configure_app_name(app_name)
           expect(config_file).to configure_push_api_key(push_api_key)
-          expect(config_file).to configure_request_headers
           expect(config_file).to_not configure_environment("development")
           expect(config_file).to_not configure_environment("staging")
           expect(config_file).to configure_environment("production")
@@ -321,7 +299,6 @@ describe Appsignal::CLI::Install do
 
             expect(output).to include_env_push_api_key(push_api_key)
             expect(output).to_not include_env_app_name
-            expect(output).to include_env_request_headers
           end
 
           it "completes the installation" do
@@ -345,7 +322,6 @@ describe Appsignal::CLI::Install do
             expect(output).to include_file_config
             expect(config_file).to configure_app_name(app_name)
             expect(config_file).to configure_push_api_key(push_api_key)
-            expect(config_file).to configure_request_headers
             expect(config_file).to configure_environment("development")
             expect(config_file).to configure_environment("staging")
             expect(config_file).to configure_environment("production")
@@ -390,7 +366,6 @@ describe Appsignal::CLI::Install do
 
             expect(output).to include_env_push_api_key(push_api_key)
             expect(output).to include_env_app_name(app_name)
-            expect(output).to include_env_request_headers
           end
 
           it "completes the installation" do
@@ -417,7 +392,6 @@ describe Appsignal::CLI::Install do
             expect(output).to include_file_config
             expect(config_file).to configure_app_name(app_name)
             expect(config_file).to configure_push_api_key(push_api_key)
-            expect(config_file).to configure_request_headers
             expect(config_file).to configure_environment("development")
             expect(config_file).to configure_environment("staging")
             expect(config_file).to configure_environment("production")
@@ -463,7 +437,6 @@ describe Appsignal::CLI::Install do
 
             expect(output).to include_env_push_api_key(push_api_key)
             expect(output).to include_env_app_name(app_name)
-            expect(output).to include_env_request_headers
           end
 
           it "completes the installation" do
@@ -487,7 +460,6 @@ describe Appsignal::CLI::Install do
             expect(output).to include_file_config
             expect(config_file).to configure_app_name(app_name)
             expect(config_file).to configure_push_api_key(push_api_key)
-            expect(config_file).to configure_request_headers
             expect(config_file).to configure_environment("development")
             expect(config_file).to configure_environment("staging")
             expect(config_file).to configure_environment("production")
@@ -532,7 +504,6 @@ describe Appsignal::CLI::Install do
 
             expect(output).to include_env_push_api_key(push_api_key)
             expect(output).to include_env_app_name(app_name)
-            expect(output).to include_env_request_headers
           end
 
           it "completes the installation" do
@@ -556,7 +527,6 @@ describe Appsignal::CLI::Install do
             expect(output).to include_file_config
             expect(config_file).to configure_app_name(app_name)
             expect(config_file).to configure_push_api_key(push_api_key)
-            expect(config_file).to configure_request_headers
             expect(config_file).to configure_environment("development")
             expect(config_file).to configure_environment("staging")
             expect(config_file).to configure_environment("production")
@@ -601,7 +571,6 @@ describe Appsignal::CLI::Install do
 
             expect(output).to include_env_push_api_key(push_api_key)
             expect(output).to include_env_app_name(app_name)
-            expect(output).to include_env_request_headers
           end
 
           it "completes the installation" do
@@ -625,7 +594,6 @@ describe Appsignal::CLI::Install do
             expect(output).to include_file_config
             expect(config_file).to configure_app_name(app_name)
             expect(config_file).to configure_push_api_key(push_api_key)
-            expect(config_file).to configure_request_headers
             expect(config_file).to configure_environment("development")
             expect(config_file).to configure_environment("staging")
             expect(config_file).to configure_environment("production")
@@ -657,7 +625,6 @@ describe Appsignal::CLI::Install do
           "\e[31mWarning:\e[0m We could not detect which framework you are using."
         expect(output).to_not include_env_push_api_key
         expect(output).to_not include_env_app_name
-        expect(output).to_not include_env_request_headers
         expect(File.exist?(config_file_path)).to be_falsy
       end
     end

data/spec/lib/appsignal/config_spec.rb

@@ -658,37 +658,5 @@ describe Appsignal::Config do
         end
       end
     end
-
-    describe "request_headers option validation" do
-      let(:out_stream) { std_stream }
-      let(:output) { out_stream.read }
-      let(:push_api_key) { "abc" }
-      let(:config_options) { { :push_api_key => push_api_key } }
-      before do
-        capture_stdout(out_stream) do
-          config.validate
-        end
-      end
-
-      context "with missing request_headers config option" do
-        let(:config_options) { { :push_api_key => "abc" } }
-
-        it "logs a warning" do
-          is_expected.to eq(true)
-          expect(output).to include "Warning: The `request_headers` config " \
-            "option was not set  in the AppSignal configuration"
-        end
-      end
-
-      context "with request_headers config option present" do
-        let(:config_options) { { :push_api_key => "abc", :request_headers => [] } }
-
-        it "logs no warning" do
-          is_expected.to eq(true)
-          expect(output).to_not include "Warning: The `request_headers` config " \
-            "option was not set  in the AppSignal configuration"
-        end
-      end
-    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 2.6.0
+  version: 2.6.1
 platform: ruby
 authors:
 - Robert Beekman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-07 00:00:00.000000000 Z
+date: 2018-05-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack