honeybadger 5.1.0 → 5.2.1

data/lib/honeybadger/util/lambda.rb

@@ -1,32 +1,32 @@
-module Honeybadger
-  module Util
-    class Lambda
-      AWS_ENV_MAP = {
-        "_HANDLER" => "handler",
-        "AWS_REGION" => "region",
-        "AWS_EXECUTION_ENV" => "runtime",
-        "AWS_LAMBDA_FUNCTION_NAME" => "function",
-        "AWS_LAMBDA_FUNCTION_MEMORY_SIZE" => "memory",
-        "AWS_LAMBDA_FUNCTION_VERSION" => "version",
-        "AWS_LAMBDA_LOG_GROUP_NAME" => "log_group",
-        "AWS_LAMBDA_LOG_STREAM_NAME" => "log_name"
-      }.freeze
-
-      class << self
-        def lambda_execution?
-          !!ENV["AWS_LAMBDA_FUNCTION_NAME"]
-        end
-
-        def normalized_data
-          AWS_ENV_MAP.each_with_object({}) do |(k, v), memo|
-            memo[v] = ENV[k] if ENV[k]
-          end
-        end
-
-        def trace_id
-          ENV["_X_AMZN_TRACE_ID"]
-        end
-      end
-    end
-  end
-end
+module Honeybadger
+  module Util
+    class Lambda
+      AWS_ENV_MAP = {
+        "_HANDLER" => "handler",
+        "AWS_REGION" => "region",
+        "AWS_EXECUTION_ENV" => "runtime",
+        "AWS_LAMBDA_FUNCTION_NAME" => "function",
+        "AWS_LAMBDA_FUNCTION_MEMORY_SIZE" => "memory",
+        "AWS_LAMBDA_FUNCTION_VERSION" => "version",
+        "AWS_LAMBDA_LOG_GROUP_NAME" => "log_group",
+        "AWS_LAMBDA_LOG_STREAM_NAME" => "log_name"
+      }.freeze
+
+      class << self
+        def lambda_execution?
+          !!ENV["AWS_LAMBDA_FUNCTION_NAME"]
+        end
+
+        def normalized_data
+          AWS_ENV_MAP.each_with_object({}) do |(k, v), memo|
+            memo[v] = ENV[k] if ENV[k]
+          end
+        end
+
+        def trace_id
+          ENV["_X_AMZN_TRACE_ID"]
+        end
+      end
+    end
+  end
+end

data/lib/honeybadger/util/request_hash.rb

@@ -1,73 +1,73 @@
-require 'set'
-
-module Honeybadger
-  module Util
-    # Constructs a request hash from a Rack::Request matching the /v1/notices
-    # API specification.
-    module RequestHash
-      HTTP_HEADER_PREFIX = 'HTTP_'.freeze
-
-      CGI_WHITELIST = %w(
-        AUTH_TYPE
-        CONTENT_LENGTH
-        CONTENT_TYPE
-        GATEWAY_INTERFACE
-        HTTPS
-        REMOTE_ADDR
-        REMOTE_HOST
-        REMOTE_IDENT
-        REMOTE_USER
-        REQUEST_METHOD
-        SERVER_NAME
-        SERVER_PORT
-        SERVER_PROTOCOL
-        SERVER_SOFTWARE
-      ).freeze
-
-      def self.from_env(env)
-        return {} unless defined?(::Rack::Request)
-        return {} unless env
-
-        hash, request = {}, ::Rack::Request.new(env.dup)
-
-        hash[:url] = extract_url(request)
-        hash[:params] = extract_params(request)
-        hash[:component] = hash[:params]['controller']
-        hash[:action] = hash[:params]['action']
-        hash[:session] = extract_session(request)
-        hash[:cgi_data] = extract_cgi_data(request)
-
-        hash
-      end
-
-      def self.extract_url(request)
-        request.env['honeybadger.request.url'] || request.url
-      rescue => e
-        "Failed to access URL -- #{e}"
-      end
-
-      def self.extract_params(request)
-        (request.env['action_dispatch.request.parameters'] || request.params).to_hash || {}
-      rescue => e
-        { error: "Failed to access params -- #{e}" }
-      end
-
-      def self.extract_session(request)
-        request.session.to_hash
-      rescue => e
-        # Rails raises ArgumentError when `config.secret_token` is missing, and
-        # ActionDispatch::Session::SessionRestoreError when the session can't be
-        # restored.
-        { error: "Failed to access session data -- #{e}" }
-      end
-
-      def self.extract_cgi_data(request)
-        request.env.each_with_object({}) do |(k,v), env|
-          next unless k.is_a?(String)
-          next unless k.start_with?(HTTP_HEADER_PREFIX) || CGI_WHITELIST.include?(k)
-          env[k] = v
-        end
-      end
-    end
-  end
-end
+require 'set'
+
+module Honeybadger
+  module Util
+    # Constructs a request hash from a Rack::Request matching the /v1/notices
+    # API specification.
+    module RequestHash
+      HTTP_HEADER_PREFIX = 'HTTP_'.freeze
+
+      CGI_WHITELIST = %w(
+        AUTH_TYPE
+        CONTENT_LENGTH
+        CONTENT_TYPE
+        GATEWAY_INTERFACE
+        HTTPS
+        REMOTE_ADDR
+        REMOTE_HOST
+        REMOTE_IDENT
+        REMOTE_USER
+        REQUEST_METHOD
+        SERVER_NAME
+        SERVER_PORT
+        SERVER_PROTOCOL
+        SERVER_SOFTWARE
+      ).freeze
+
+      def self.from_env(env)
+        return {} unless defined?(::Rack::Request)
+        return {} unless env
+
+        hash, request = {}, ::Rack::Request.new(env.dup)
+
+        hash[:url] = extract_url(request)
+        hash[:params] = extract_params(request)
+        hash[:component] = hash[:params]['controller']
+        hash[:action] = hash[:params]['action']
+        hash[:session] = extract_session(request)
+        hash[:cgi_data] = extract_cgi_data(request)
+
+        hash
+      end
+
+      def self.extract_url(request)
+        request.env['honeybadger.request.url'] || request.url
+      rescue => e
+        "Failed to access URL -- #{e}"
+      end
+
+      def self.extract_params(request)
+        (request.env['action_dispatch.request.parameters'] || request.params).to_hash || {}
+      rescue => e
+        { error: "Failed to access params -- #{e}" }
+      end
+
+      def self.extract_session(request)
+        request.session.to_hash
+      rescue => e
+        # Rails raises ArgumentError when `config.secret_token` is missing, and
+        # ActionDispatch::Session::SessionRestoreError when the session can't be
+        # restored.
+        { error: "Failed to access session data -- #{e}" }
+      end
+
+      def self.extract_cgi_data(request)
+        request.env.each_with_object({}) do |(k,v), env|
+          next unless k.is_a?(String)
+          next unless k.start_with?(HTTP_HEADER_PREFIX) || CGI_WHITELIST.include?(k)
+          env[k] = v
+        end
+      end
+    end
+  end
+end

data/lib/honeybadger/util/request_payload.rb

@@ -1,41 +1,41 @@
-require 'honeybadger/util/sanitizer'
-
-module Honeybadger
-  module Util
-    # Constructs/sanitizes request data for notices
-    module RequestPayload
-      # Default values to use for request data.
-      DEFAULTS = {
-        url: nil,
-        component: nil,
-        action: nil,
-        params: {}.freeze,
-        session: {}.freeze,
-        cgi_data: {}.freeze
-      }.freeze
-
-      # Allowed keys.
-      KEYS = DEFAULTS.keys.freeze
-
-      # The cgi_data key where the raw Cookie header is stored.
-      HTTP_COOKIE_KEY = 'HTTP_COOKIE'.freeze
-
-      def self.build(opts = {})
-        sanitizer = opts.fetch(:sanitizer) { Sanitizer.new }
-
-        payload = DEFAULTS.dup
-        KEYS.each do |key|
-          next unless opts[key]
-          payload[key] = sanitizer.sanitize(opts[key])
-        end
-
-        payload[:url] = sanitizer.filter_url(payload[:url]) if payload[:url]
-        if payload[:cgi_data][HTTP_COOKIE_KEY]
-          payload[:cgi_data][HTTP_COOKIE_KEY] = sanitizer.filter_cookies(payload[:cgi_data][HTTP_COOKIE_KEY])
-        end
-
-        payload
-      end
-    end
-  end
-end
+require 'honeybadger/util/sanitizer'
+
+module Honeybadger
+  module Util
+    # Constructs/sanitizes request data for notices
+    module RequestPayload
+      # Default values to use for request data.
+      DEFAULTS = {
+        url: nil,
+        component: nil,
+        action: nil,
+        params: {}.freeze,
+        session: {}.freeze,
+        cgi_data: {}.freeze
+      }.freeze
+
+      # Allowed keys.
+      KEYS = DEFAULTS.keys.freeze
+
+      # The cgi_data key where the raw Cookie header is stored.
+      HTTP_COOKIE_KEY = 'HTTP_COOKIE'.freeze
+
+      def self.build(opts = {})
+        sanitizer = opts.fetch(:sanitizer) { Sanitizer.new }
+
+        payload = DEFAULTS.dup
+        KEYS.each do |key|
+          next unless opts[key]
+          payload[key] = sanitizer.sanitize(opts[key])
+        end
+
+        payload[:url] = sanitizer.filter_url(payload[:url]) if payload[:url]
+        if payload[:cgi_data][HTTP_COOKIE_KEY]
+          payload[:cgi_data][HTTP_COOKIE_KEY] = sanitizer.filter_cookies(payload[:cgi_data][HTTP_COOKIE_KEY])
+        end
+
+        payload
+      end
+    end
+  end
+end

data/lib/honeybadger/util/revision.rb

@@ -1,39 +1,39 @@
-module Honeybadger
-  module Util
-    class Revision
-      class << self
-        def detect(root = Dir.pwd)
-          revision = from_heroku ||
-            from_capistrano(root) ||
-            from_git
-
-          revision = revision.to_s.strip
-          return unless revision =~ /\S/
-
-          revision
-        end
-
-        private
-
-        # Requires (currently) alpha platform feature `heroku labs:enable
-        # runtime-dyno-metadata`
-        #
-        # See https://devcenter.heroku.com/articles/dyno-metadata
-        def from_heroku
-          ENV['HEROKU_SLUG_COMMIT']
-        end
-
-        def from_capistrano(root)
-          file = File.join(root, 'REVISION')
-          return nil unless File.file?(file)
-          File.read(file) rescue nil
-        end
-
-        def from_git
-          return nil unless File.directory?('.git')
-          `git rev-parse HEAD 2> #{File::NULL}` rescue nil
-        end
-      end
-    end
-  end
-end
+module Honeybadger
+  module Util
+    class Revision
+      class << self
+        def detect(root = Dir.pwd)
+          revision = from_heroku ||
+            from_capistrano(root) ||
+            from_git
+
+          revision = revision.to_s.strip
+          return unless revision =~ /\S/
+
+          revision
+        end
+
+        private
+
+        # Requires (currently) alpha platform feature `heroku labs:enable
+        # runtime-dyno-metadata`
+        #
+        # See https://devcenter.heroku.com/articles/dyno-metadata
+        def from_heroku
+          ENV['HEROKU_SLUG_COMMIT']
+        end
+
+        def from_capistrano(root)
+          file = File.join(root, 'REVISION')
+          return nil unless File.file?(file)
+          File.read(file) rescue nil
+        end
+
+        def from_git
+          return nil unless File.directory?('.git')
+          `git rev-parse HEAD 2> #{File::NULL}` rescue nil
+        end
+      end
+    end
+  end
+end