exception_notification 4.4.0 → 4.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: f61027914f50eed12ef3c115cca8ba7a3f4f3b1e
-  data.tar.gz: 1a63465348250221f3a7d2171979a8d6b35639c1
+SHA256:
+  metadata.gz: f671fa31cec6dbf437c5e31ab10136a00f3ef5450fc13d9de2887296390451ba
+  data.tar.gz: 20f146f16e31e91c456cb08f5598b2714dec47ef1efc41741c7cfea761a8176f
 SHA512:
-  metadata.gz: 62cf62e3500f7edb53940bbe458a5b2e1936469033c4c643ad47d4ac9d3a913bd406fb7ca37cbf0d27b4f862b5487bd5736abdd37288aaaa904a125a00bd72bf
-  data.tar.gz: 0ffc513a9fd0ce553254665abfb6da9640a4e9eb3ff5edf39aa838b9baa3caffffd76f0ffab64417721cd7c4828fdabba8c6ab9c9f47077f800845418c38db52
+  metadata.gz: a3c4f84bef2faccc2a66306349b983843a1542895d13e8cb141e50e8c610dc525087f8f86fc4c40654140e128c5378b224851c888086b0b32a6cb4753011717e
+  data.tar.gz: 5aa30cabeb47f39f89d13c904bdd36da33bf7a598cdb24f14ff6882e90826e3985b1bfb19c55a250bdfc3edae343ec56a72887a5bdbd195bbe4251738a65d1b2

data/Appraisals

@@ -1,4 +1,6 @@
-rails_versions = ['~> 4.0.5', '~> 4.1.1', '~> 4.2.0', '~> 5.0.0', '~> 5.1.0', '~> 5.2.0']
+# frozen_string_literal: true
+
+rails_versions = ['~> 5.2.0', '~> 6.0.0', '~> 6.1.0', '~> 7.0.0']
 
 rails_versions.each do |rails_version|
   appraise "rails#{rails_version.slice(/\d+\.\d+/).tr('.', '_')}" do

data/CHANGELOG.rdoc

@@ -1,3 +1,36 @@
+== 4.5.0
+
+* enhancements
+  * Added Rails 7 compatibility (by @fwininger)
+  * Added support for the optional `data` attribute to the SNS notifier (@TomK32)
+  * Addressed a deprecation warning for `module_parent_name` which was thrown for users
+    using Rails > 6.x (@quorak)
+  * Restored the hash separator for `controller#action` in the email notifier (@garethrees)
+
+* removals
+  * Dropped support for Tinder (gem is no longer maintained) (by @fwininger)
+  * Dropped support for Ruby on Rails versions below 5.2
+
+== 4.4.3
+
+* big fixes
+  * Remove using configured default from address from custom mailer_parent class
+
+== 4.4.2 (yanked)
+
+* bug fixes
+  * Fix `sender_address` being overwritten
+
+== 4.4.1
+
+* enhancements
+  * Enhance `ignore_if` option to allow by-notifier customization (by @fursich)
+  * Ignore extended modules of ignored exceptions (by @elengine)
+  * Add `exception_data` to Mattermost notifier (by @camillof)
+
+* bug fixes
+  * Fix Rubocop offenses (by @nicolasferraro)
+
 == 4.4.0
 
 * enhancements

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gemspec

data/README.md

@@ -1,15 +1,15 @@
 # Exception Notification
 
-[![Gem Version](https://fury-badge.herokuapp.com/rb/exception_notification.png)](http://badge.fury.io/rb/exception_notification)
-[![Travis](https://api.travis-ci.org/smartinez87/exception_notification.png)](http://travis-ci.org/smartinez87/exception_notification)
-[![Coverage Status](https://coveralls.io/repos/smartinez87/exception_notification/badge.png?branch=master)](https://coveralls.io/r/smartinez87/exception_notification)
-[![Code Climate](https://codeclimate.com/github/smartinez87/exception_notification.png)](https://codeclimate.com/github/smartinez87/exception_notification)
+[![Gem Version](https://badge.fury.io/rb/exception_notification.svg)](https://badge.fury.io/rb/exception_notification)
+[![Build Status](https://github.com/smartinez87/exception_notification/actions/workflows/main.yml/badge.svg)](https://github.com/smartinez87/exception_notification/actions/workflows/main.yml)
+[![Coverage Status](https://coveralls.io/repos/github/smartinez87/exception_notification/badge.svg?branch=master)](https://coveralls.io/github/smartinez87/exception_notification?branch=master)
+[![Maintainability](https://api.codeclimate.com/v1/badges/78a9a12be00a6d305136/maintainability)](https://codeclimate.com/github/smartinez87/exception_notification/maintainability)
 
 **THIS README IS FOR THE MASTER BRANCH AND REFLECTS THE WORK CURRENTLY EXISTING ON THE MASTER BRANCH. IF YOU ARE WISHING TO USE A NON-MASTER BRANCH OF EXCEPTION NOTIFICATION, PLEASE CONSULT THAT BRANCH'S README AND NOT THIS ONE.**
 
 ---
 
-The Exception Notification gem provides a set of [notifiers](#notifiers) for sending notifications when errors occur in a Rack/Rails application. The built-in notifiers can deliver notifications by [email](docs/notifiers/email.md), [Campfire](docs/notifiers/campfire.md), [HipChat](docs/notifiers/hipchat.md), [Slack](docs/notifiers/slack.md), [Mattermost](docs/notifiers/mattermost.md), [Teams](docs/notifiers/teams.md), [IRC](docs/notifiers/irc.md), [Amazon SNS](docs/notifiers/sns.md), [Google Chat](docs/notifiers/google_chat.md), [Datadog](docs/notifiers/datadog.md) or via custom [WebHooks](docs/notifiers/webhook.md).
+The Exception Notification gem provides a set of [notifiers](#notifiers) for sending notifications when errors occur in a Rack/Rails application. The built-in notifiers can deliver notifications by [email](docs/notifiers/email.md), [HipChat](docs/notifiers/hipchat.md), [Slack](docs/notifiers/slack.md), [Mattermost](docs/notifiers/mattermost.md), [Teams](docs/notifiers/teams.md), [IRC](docs/notifiers/irc.md), [Amazon SNS](docs/notifiers/sns.md), [Google Chat](docs/notifiers/google_chat.md), [Datadog](docs/notifiers/datadog.md) or via custom [WebHooks](docs/notifiers/webhook.md).
 
 There's a great [Railscast about Exception Notification](http://railscasts.com/episodes/104-exception-notifications-revised) you can see that may help you getting started.
 
@@ -17,10 +17,8 @@ There's a great [Railscast about Exception Notification](http://railscasts.com/e
 
 ## Requirements
 
-* Ruby 2.0 or greater
-* Rails 4.0 or greater, Sinatra or another Rack-based application.
-
-For previous releases, please checkout [this](#versions).
+* Ruby 2.5 or greater
+* Rails 5.2 or greater, Sinatra or another Rack-based application.
 
 ## Getting Started
 
@@ -37,7 +35,6 @@ ExceptionNotification is used as a rack middleware, or in the environment you wa
 ```ruby
 Rails.application.config.middleware.use ExceptionNotification::Rack,
   email: {
-    deliver_with: :deliver, # Rails >= 4.2.1 do not need this option since it defaults to :deliver_now
     email_prefix: '[PREFIX] ',
     sender_address: %{"notifier" <notifier@example.com>},
     exception_recipients: %w{exceptions@example.com}
@@ -86,7 +83,6 @@ Options -> sections" below.
 
 ExceptionNotification relies on notifiers to deliver notifications when errors occur in your applications. By default, 8 notifiers are available:
 
-* [Campfire notifier](docs/notifiers/campfire.md)
 * [Datadog notifier](docs/notifiers/datadog.md)
 * [Email notifier](docs/notifiers/email.md)
 * [HipChat notifier](docs/notifiers/hipchat.md)
@@ -138,6 +134,8 @@ You can choose to ignore certain exceptions, which will make ExceptionNotificati
 
 * `:ignore_if`         - Custom (i.e. ignore exceptions that satisfy some condition)
 
+* `:ignore_notifer_if` - Custom (i.e. let each notifier ignore exceptions if by-notifier condition is satisfied)
+
 
 ### :ignore_exceptions
 
@@ -177,7 +175,7 @@ Rails.application.config.middleware.use ExceptionNotification::Rack,
 
 *Lambda, default: nil*
 
-Last but not least, you can ignore exceptions based on a condition. Take a look:
+You can ignore exceptions based on a condition. Take a look:
 
 ```ruby
 Rails.application.config.middleware.use ExceptionNotification::Rack,
@@ -191,6 +189,32 @@ Rails.application.config.middleware.use ExceptionNotification::Rack,
 
 You can make use of both the environment and the exception inside the lambda to decide wether to avoid or not sending the notification.
 
+### :ignore_notifier_if
+
+* Hash of Lambda, default: nil*
+
+In case you want a notifier to ignore certain exceptions, but don't want other notifiers to skip them, you can set by-notifier ignore options.
+By setting below, each notifier will ignore exceptions when its corresponding condition is met.
+
+```ruby
+Rails.application.config.middleware.use ExceptionNotification::Rack,
+                                        ignore_notifier_if: {
+                                          email: ->(env, exception) { !Rails.env.production? },
+                                          slack: ->(env, exception) { exception.message =~ /^Couldn't find Page with ID=/ }
+                                        }
+
+                                        email: {
+                                          sender_address: %{"notifier" <notifier@example.com>},
+                                          exception_recipients: %w{exceptions@example.com}
+                                        },
+                                        slack: {
+                                          webhook_url: '[Your webhook url]',
+                                          channel: '#exceptions',
+                                        }
+```
+
+To customize each condition, you can make use of environment and the exception object inside the lambda.
+
 ## Rack X-Cascade Header
 
 Some rack apps (Rails in particular) utilize the "X-Cascade" header to pass the request-handling responsibility to the next middleware in the stack.
@@ -245,7 +269,6 @@ def server_error(exception)
 end
 ```
 
-
 ## Extras
 
 ### Rails

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems'
 require 'bundler/setup'
 Bundler::GemHelper.install_tasks

data/examples/sample_app.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # -------------------------------------------
 # To run the application: ruby examples/sample_app.rb
 # -------------------------------------------

data/examples/sinatra/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gem 'exception_notification', path: '../../'

data/examples/sinatra/config.ru

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require ::File.expand_path('../sinatra_app', __FILE__)
 
 run SinatraApp

data/examples/sinatra/sinatra_app.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems'
 require 'bundler/setup'
 require 'sinatra/base'
@@ -5,7 +7,8 @@ require 'exception_notification'
 
 class SinatraApp < Sinatra::Base
   use Rack::Config do |env|
-    env['action_dispatch.parameter_filter'] = [:password] # This is highly recommended.  It will prevent the ExceptionNotification email from including your users' passwords
+    # This is highly recommended.  It will prevent the ExceptionNotification email from including your users' passwords
+    env['action_dispatch.parameter_filter'] = [:password]
   end
 
   use ExceptionNotification::Rack,
@@ -22,7 +25,8 @@ class SinatraApp < Sinatra::Base
   get '/' do
     raise StandardError, "ERROR: #{params[:error]}" unless params[:error].blank?
 
-    'Everything is fine! Now, lets break things clicking <a href="/?error=ops"> here </a>. Dont forget to see the emails at <a href="http://localhost:1080">mailcatcher</a> !'
+    'Everything is fine! Now, lets break things clicking <a href="/?error=ops"> here </a>.' \
+    'Dont forget to see the emails at <a href="http://localhost:1080">mailcatcher</a> !'
   end
 
   get '/background_notification' do

data/exception_notification.gemspec

@@ -1,16 +1,18 @@
-require File.expand_path('../lib/exception_notification/version', __FILE__)
+# frozen_string_literal: true
+
+require File.expand_path('lib/exception_notification/version', __dir__)
 
 Gem::Specification.new do |s|
   s.name = 'exception_notification'
   s.version = ExceptionNotification::VERSION
   s.authors = ['Jamis Buck', 'Josh Peek']
-  s.date = '2019-08-16'
+  s.date = '2022-01-20'
   s.summary = 'Exception notification for Rails apps'
   s.homepage = 'https://smartinez87.github.io/exception_notification/'
   s.email = 'smartinez87@gmail.com'
   s.license = 'MIT'
 
-  s.required_ruby_version     = '>= 2.0'
+  s.required_ruby_version     = '>= 2.3'
   s.required_rubygems_version = '>= 1.8.11'
 
   s.files = `git ls-files`.split("\n")
@@ -18,8 +20,8 @@ Gem::Specification.new do |s|
   s.test_files = `git ls-files -- test`.split("\n")
   s.require_path = 'lib'
 
-  s.add_dependency('actionmailer', '>= 4.0', '< 7')
-  s.add_dependency('activesupport', '>= 4.0', '< 7')
+  s.add_dependency('actionmailer', '>= 5.2', '< 8')
+  s.add_dependency('activesupport', '>= 5.2', '< 8')
 
   s.add_development_dependency 'appraisal', '~> 2.2.0'
   s.add_development_dependency 'aws-sdk-sns', '~> 1'
@@ -28,14 +30,13 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'dogapi', '>= 1.23.0'
   s.add_development_dependency 'hipchat', '>= 1.0.0'
   s.add_development_dependency 'httparty', '~> 0.10.2'
-  s.add_development_dependency 'mock_redis', '~> 0.18.0'
   s.add_development_dependency 'mocha', '>= 0.13.0'
-  s.add_development_dependency 'rails', '>= 4.0', '< 7'
+  s.add_development_dependency 'mock_redis', '~> 0.19.0'
+  s.add_development_dependency 'net-smtp'
+  s.add_development_dependency 'rails', '>= 5.2', '< 8'
   s.add_development_dependency 'resque', '~> 1.8.0'
-  s.add_development_dependency 'rubocop', '0.50.0'
-  # Sidekiq 3.2.2 does not support Ruby 1.9.
-  s.add_development_dependency 'sidekiq', '~> 3.0.0', '< 3.2.2'
+  s.add_development_dependency 'rubocop', '0.78.0'
+  s.add_development_dependency 'sidekiq', '>= 5.0.4'
   s.add_development_dependency 'slack-notifier', '>= 1.0.0'
-  s.add_development_dependency 'timecop', '~>0.9.0'
-  s.add_development_dependency 'tinder', '~> 1.8'
+  s.add_development_dependency 'timecop', '~> 0.9.0'
 end

data/gemfiles/rails5_2.gemfile

@@ -1,7 +1,7 @@
 # This file was generated by Appraisal
 
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
-gem 'rails', '~> 5.2.0'
+gem "rails", "~> 5.2.0"
 
-gemspec path: '../'
+gemspec path: "../"

data/gemfiles/rails6_1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 6.1.0"
+
+gemspec path: "../"

data/gemfiles/rails7_0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 7.0.0"
+
+gemspec path: "../"

data/lib/exception_notification/rack.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotification
   class Rack
     class CascadePassException < RuntimeError; end
@@ -5,15 +7,17 @@ module ExceptionNotification
     def initialize(app, options = {})
       @app = app
 
-      ExceptionNotifier.ignored_exceptions = options.delete(:ignore_exceptions) if options.key?(:ignore_exceptions)
-      ExceptionNotifier.error_grouping = options.delete(:error_grouping) if options.key?(:error_grouping)
-      ExceptionNotifier.error_grouping_period = options.delete(:error_grouping_period) if options.key?(:error_grouping_period)
-      ExceptionNotifier.notification_trigger = options.delete(:notification_trigger) if options.key?(:notification_trigger)
+      ExceptionNotifier.tap do |en|
+        en.ignored_exceptions = options.delete(:ignore_exceptions) if options.key?(:ignore_exceptions)
+        en.error_grouping = options.delete(:error_grouping) if options.key?(:error_grouping)
+        en.error_grouping_period = options.delete(:error_grouping_period) if options.key?(:error_grouping_period)
+        en.notification_trigger = options.delete(:notification_trigger) if options.key?(:notification_trigger)
 
-      if options.key?(:error_grouping_cache)
-        ExceptionNotifier.error_grouping_cache = options.delete(:error_grouping_cache)
-      elsif defined?(Rails) && Rails.respond_to?(:cache)
-        ExceptionNotifier.error_grouping_cache = Rails.cache
+        if options.key?(:error_grouping_cache)
+          en.error_grouping_cache = options.delete(:error_grouping_cache)
+        elsif defined?(Rails) && Rails.respond_to?(:cache)
+          en.error_grouping_cache = Rails.cache
+        end
       end
 
       if options.key?(:ignore_if)
@@ -23,6 +27,15 @@ module ExceptionNotification
         end
       end
 
+      if options.key?(:ignore_notifier_if)
+        rack_ignore_by_notifier = options.delete(:ignore_notifier_if)
+        rack_ignore_by_notifier.each do |notifier, proc|
+          ExceptionNotifier.ignore_notifier_if(notifier) do |exception, opts|
+            opts.key?(:env) && proc.call(opts[:env], exception)
+          end
+        end
+      end
+
       ExceptionNotifier.ignore_crawlers(options.delete(:ignore_crawlers)) if options.key?(:ignore_crawlers)
 
       @ignore_cascade_pass = options.delete(:ignore_cascade_pass) { true }
@@ -42,12 +55,10 @@ module ExceptionNotification
       end
 
       response
-    rescue Exception => exception
-      if ExceptionNotifier.notify_exception(exception, env: env)
-        env['exception_notifier.delivered'] = true
-      end
+    rescue Exception => e
+      env['exception_notifier.delivered'] = true if ExceptionNotifier.notify_exception(e, env: env)
 
-      raise exception unless exception.is_a?(CascadePassException)
+      raise e unless e.is_a?(CascadePassException)
 
       response
     end

data/lib/exception_notification/rails.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotification
   class Engine < ::Rails::Engine
     config.exception_notification = ExceptionNotifier

data/lib/exception_notification/resque.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'resque/failure/base'
 
 module ExceptionNotification

data/lib/exception_notification/sidekiq.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'sidekiq'
 
 # Note: this class is only needed for Sidekiq version < 3.
@@ -5,9 +7,9 @@ module ExceptionNotification
   class Sidekiq
     def call(_worker, msg, _queue)
       yield
-    rescue Exception => exception
-      ExceptionNotifier.notify_exception(exception, data: { sidekiq: msg })
-      raise exception
+    rescue Exception => e
+      ExceptionNotifier.notify_exception(e, data: { sidekiq: msg })
+      raise e
     end
   end
 end

data/lib/exception_notification/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotification
-  VERSION = '4.4.0'.freeze
+  VERSION = '4.5.0'
 end

data/lib/exception_notification.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'exception_notifier'
 require 'exception_notification/rack'
 require 'exception_notification/version'

data/lib/exception_notifier/base_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotifier
   class BaseNotifier
     attr_accessor :base_options
@@ -14,11 +16,15 @@ module ExceptionNotifier
     end
 
     def _pre_callback(exception, options, message, message_opts)
-      @base_options[:pre_callback].call(options, self, exception.backtrace, message, message_opts) if @base_options[:pre_callback].respond_to?(:call)
+      return unless @base_options[:pre_callback].respond_to?(:call)
+
+      @base_options[:pre_callback].call(options, self, exception.backtrace, message, message_opts)
     end
 
     def _post_callback(exception, options, message, message_opts)
-      @base_options[:post_callback].call(options, self, exception.backtrace, message, message_opts) if @base_options[:post_callback].respond_to?(:call)
+      return unless @base_options[:post_callback].respond_to?(:call)
+
+      @base_options[:post_callback].call(options, self, exception.backtrace, message, message_opts)
     end
   end
 end

data/lib/exception_notifier/datadog_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'action_dispatch'
 
 module ExceptionNotifier
@@ -30,7 +32,7 @@ module ExceptionNotifier
       MAX_TITLE_LENGTH = 120
       MAX_VALUE_LENGTH = 300
       MAX_BACKTRACE_SIZE = 3
-      ALERT_TYPE = 'error'.freeze
+      ALERT_TYPE = 'error'
 
       attr_reader :exception,
                   :options
@@ -74,11 +76,8 @@ module ExceptionNotifier
       end
 
       def formatted_title
-        title = ''
-        title << title_prefix
-        title << "#{controller.controller_name} #{controller.action_name}" if controller
-        title << " (#{exception.class})"
-        title << " #{exception.message.inspect}"
+        title =
+          "#{title_prefix}#{controller_subtitle} (#{exception.class}) #{exception.message.inspect}"
 
         truncate(title, MAX_TITLE_LENGTH)
       end
@@ -108,9 +107,7 @@ module ExceptionNotifier
         text << formatted_key_value('Parameters', request.filtered_parameters.inspect)
         text << formatted_key_value('Timestamp', Time.current)
         text << formatted_key_value('Server', Socket.gethostname)
-        if defined?(Rails) && Rails.respond_to?(:root)
-          text << formatted_key_value('Rails root', Rails.root)
-        end
+        text << formatted_key_value('Rails root', Rails.root) if defined?(Rails) && Rails.respond_to?(:root)
         text << formatted_key_value('Process', $PROCESS_ID)
         text << '___'
         text.join("\n")
@@ -148,6 +145,12 @@ module ExceptionNotifier
           object.to_s
         end
       end
+
+      private
+
+      def controller_subtitle
+        "#{controller.controller_name} #{controller.action_name}" if controller
+      end
     end
   end
 end

data/lib/exception_notifier/email_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/core_ext/time'
 require 'action_mailer'
 require 'action_dispatch'
@@ -74,13 +76,17 @@ module ExceptionNotifier
           def compose_subject
             subject = @options[:email_prefix].to_s.dup
             subject << "(#{@options[:accumulated_errors_count]} times)" if @options[:accumulated_errors_count].to_i > 1
-            subject << "#{@kontroller.controller_name} #{@kontroller.action_name}" if @kontroller && @options[:include_controller_and_action_names_in_subject]
+            subject << "#{@kontroller.controller_name}##{@kontroller.action_name}" if include_controller?
             subject << " (#{@exception.class})"
             subject << " #{@exception.message.inspect}" if @options[:verbose_subject]
             subject = EmailNotifier.normalize_digits(subject) if @options[:normalize_subject]
             subject.length > 120 ? subject[0...120] + '...' : subject
           end
 
+          def include_controller?
+            @kontroller && @options[:include_controller_and_action_names_in_subject]
+          end
+
           def set_data_variables
             @data.each do |name, value|
               instance_variable_set("@#{name}", value)
@@ -137,7 +143,9 @@ module ExceptionNotifier
           end
 
           def load_custom_views
-            prepend_view_path Rails.root.nil? ? 'app/views' : "#{Rails.root}/app/views" if defined?(Rails) && Rails.respond_to?(:root)
+            return unless defined?(Rails) && Rails.respond_to?(:root)
+
+            prepend_view_path Rails.root.nil? ? 'app/views' : "#{Rails.root}/app/views"
           end
 
           def maybe_call(maybe_proc)

data/lib/exception_notifier/google_chat_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'httparty'
 
 module ExceptionNotifier

data/lib/exception_notifier/hipchat_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotifier
   class HipchatNotifier < BaseNotifier
     attr_accessor :from

data/lib/exception_notifier/irc_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotifier
   class IrcNotifier < BaseNotifier
     def initialize(options)
@@ -9,9 +11,8 @@ module ExceptionNotifier
     def call(exception, options = {})
       errors_count = options[:accumulated_errors_count].to_i
 
-      message = "'#{exception.message}'"
-      message.prepend("(#{errors_count} times)") if errors_count > 1
-
+      occurrences = "(#{errors_count} times)" if errors_count > 1
+      message = "#{occurrences}'#{exception.message}'"
       message += " on '#{exception.backtrace.first}'" if exception.backtrace
 
       return unless active?

data/lib/exception_notifier/mattermost_notifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'httparty'
 
 module ExceptionNotifier
@@ -10,6 +12,8 @@ module ExceptionNotifier
 
       @gitlab_url = options[:git_url]
 
+      @env = options[:env] || {}
+
       payload = {
         text: message_text.compact.join("\n"),
         username: options[:username] || 'Exception Notifier'
@@ -52,6 +56,12 @@ module ExceptionNotifier
         text << backtrace
       end
 
+      if (exception_data = @env['exception_notifier.exception_data'])
+        text << '### Data'
+        data_string = exception_data.map { |k, v| "* #{k} : #{v}" }.join("\n")
+        text << "```\n#{data_string}\n```"
+      end
+
       text << message_issue_link if @gitlab_url
 
       text

data/lib/exception_notifier/modules/backtrace_cleaner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ExceptionNotifier
   module BacktraceCleaner
     def clean_backtrace(exception)