exception_handling 2.13.0 → 2.14.0.pre.gk.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1125ae1526cff9890e52497ac1a8b8d93a62a481afa49a74709c5f286f53b912
-  data.tar.gz: 3f882364adfb48ee837fc178cb3adcb85ab6c43b90bde5e04f96aea07559efc6
+  metadata.gz: d1f9171a98555a6b8c48908ca1c26f6c9d9f05e966030d0f7c62598378c955b4
+  data.tar.gz: ca9e28ace18d2dd3db9abd173184c638eac5540873cdcabe5b267813ca276909
 SHA512:
-  metadata.gz: efbe4b3c2843a4e07daf38a83606d504714b1bfa2dedc406070fc6a1e68a66ccf379cb459f5364c0c8c8ac43b9265474ab50d9dce7f31ec78de5791ff166c195
-  data.tar.gz: 1a69645d781332b9e7882921dbaded4d001c87ead47a40902b55e385c21e419ed48d9c3790cb38c38392a0fe2c872e05d003814bcd57b54e6cb27b38c3828772
+  metadata.gz: a9bf483a26054a430fb78a08e1eaf9bd963ea17da9e4df422c7ef25e8033a355d4e2c6296a552a903a2c965c42f32d16b8734d047bb2739fb14f0b84e9da6c78
+  data.tar.gz: b39e222737fb25d12e8070e693f7b092e6f2841f0d1eefdfa728e6ba0b3f17d825480b499c283c6dcb282aa6e94d16fdf9cddf3a32e42d2e6fab483dd838e625

data/CHANGELOG.md

@@ -4,6 +4,10 @@ Inspired by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.14.0] - 2022-12-23
+### Added
+- Added support for plumbing tags through to honeybadger via the `honeybadger_tags` log context parameter
+
 ## [2.13.0] - 2022-09-15
 ### Added
 - Added an option for removing the 'exception_handling.' prefix from metric names in ExceptionHandling::default_metric_name
@@ -99,6 +103,7 @@ Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0
 ### Changed
 - No longer depends on hobo_support. Uses invoca-utils 0.3 instead.
 
+[2.14.0]: https://github.com/Invoca/exception_handling/compare/v2.13.0...v2.14.0
 [2.13.0]: https://github.com/Invoca/exception_handling/compare/v2.12.0...v2.13.0
 [2.12.0]: https://github.com/Invoca/exception_handling/compare/v2.11.3...v2.12.0
 [2.11.3]: https://github.com/Invoca/exception_handling/compare/v2.11.2...v2.11.3

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    exception_handling (2.13.0)
+    exception_handling (2.14.0.pre.gk.0)
       actionmailer (>= 5.2, < 7.0)
       actionpack (>= 5.2, < 7.0)
       activesupport (>= 5.2, < 7.0)

data/lib/exception_handling/exception_info.rb

@@ -49,7 +49,7 @@ module ExceptionHandling
     HONEYBADGER_CONTEXT_SECTIONS = [:timestamp, :error_class, :exception_context, :server, :scm_revision, :notes,
                                     :user_details, :request, :session, :environment, :backtrace, :event_response, :log_context].freeze
 
-    attr_reader :exception, :controller, :exception_context, :timestamp
+    attr_reader :exception, :controller, :exception_context, :timestamp, :honeybadger_tags
 
     def initialize(exception, exception_context, timestamp, controller: nil, data_callback: nil, log_context: nil)
       @exception = exception
@@ -59,6 +59,7 @@ module ExceptionHandling
       @data_callback = data_callback
       # merge into the surrounding context just like ContextualLogger does when logging
       @merged_log_context = ExceptionHandling.logger.current_context_for_thread.deep_merge(log_context || {})
+      @honeybadger_tags = [@merged_log_context[:honeybadger_tags]].flatten.compact
     end
 
     def data

data/lib/exception_handling/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ExceptionHandling
-  VERSION = '2.13.0'
+  VERSION = '2.14.0.pre.gk.0'
 end

data/lib/exception_handling.rb

@@ -274,7 +274,8 @@ module ExceptionHandling # never included
                                     error_message: exception.message.to_s,
                                     exception:     exception,
                                     context:       exception_info.honeybadger_context_data,
-                                    controller:    exception_info.controller_name)
+                                    controller:    exception_info.controller_name,
+                                    tags:          exception_info.honeybadger_tags.join(', '))
       response ? :success : :failure
     rescue Exception => ex
       warn("ExceptionHandling.send_exception_to_honeybadger rescued exception while logging #{exception_info.exception_context}:\n#{exception.class}: #{exception.message}:\n#{ex.class}: #{ex.message}\n#{ex.backtrace.join("\n")}")

data/spec/unit/exception_handling/exception_info_spec.rb

@@ -44,6 +44,23 @@ module ExceptionHandling
           expect(exception_info.controller).to be_nil
         end
       end
+
+      context "honeybadger_tags" do
+        it "retrieved from log_context" do
+          exception_info = ExceptionInfo.new(@exception, "string context", @timestamp, log_context: { honeybadger_tags: ['Data Services'] })
+          expect(exception_info.honeybadger_tags).to eq(['Data Services'])
+        end
+
+        it "retrieved from log_context and normalized to array if a string" do
+          exception_info = ExceptionInfo.new(@exception, "string context", @timestamp, log_context: { honeybadger_tags: 'Data Services' })
+          expect(exception_info.honeybadger_tags).to eq(['Data Services'])
+        end
+
+        it "defaults to empty array if nil" do
+          exception_info = ExceptionInfo.new(@exception, "string context", @timestamp)
+          expect(exception_info.honeybadger_tags).to eq([])
+        end
+      end
     end
 
     context "data" do

data/spec/unit/exception_handling_spec.rb

@@ -727,7 +727,7 @@ describe ExceptionHandling do
               expect(Honeybadger).to receive(:notify).with(any_args) do |data|
                 honeybadger_data = data
               end
-              ExceptionHandling.logger.global_context = { service_name: "rails", region: "AWS-us-east-1" }
+              ExceptionHandling.logger.global_context = { service_name: "rails", region: "AWS-us-east-1", honeybadger_tags: ['Data Services', 'web'] }
               log_context = { log_source: "gem/listen", service_name: "bin/console" }
               ExceptionHandling.log_error(@exception, @exception_context, @controller, **log_context) do |data|
                 data[:scm_revision] = "5b24eac37aaa91f5784901e9aabcead36fd9df82"
@@ -741,6 +741,7 @@ describe ExceptionHandling do
                 error_message: "Some Exception",
                 controller: "some_controller",
                 exception: @exception,
+                tags: "Data Services, web",
                 context: {
                   timestamp: Time.now.to_i,
                   error_class: "StandardError",
@@ -766,7 +767,7 @@ describe ExceptionHandling do
                     "spec/unit/exception_handling_spec.rb:455:in `block (4 levels) in <class:ExceptionHandlingTest>'"
                   ],
                   event_response: "Event successfully received",
-                  log_context: { "service_name" => "bin/console", "region" => "AWS-us-east-1", "log_source" => "gem/listen" }
+                  log_context: { "service_name" => "bin/console", "region" => "AWS-us-east-1", "log_source" => "gem/listen", "honeybadger_tags" => ['Data Services', 'web'] }
                 }
               }
               expect(honeybadger_data).to eq(expected_data)
@@ -791,6 +792,7 @@ describe ExceptionHandling do
                 error_message: "Some Exception",
                 controller: "some_controller",
                 exception: @exception,
+                tags: "",
                 context: {
                   timestamp: Time.now.to_i,
                   error_class: "StandardError",

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exception_handling
 version: !ruby/object:Gem::Version
-  version: 2.13.0
+  version: 2.14.0.pre.gk.0
 platform: ruby
 authors:
 - Invoca
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-15 00:00:00.000000000 Z
+date: 2022-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionmailer
@@ -222,9 +222,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.1.6
 signing_key: