[improvement] Added configuration option for ignoring bounce notifications for certain emails (10m)

Should hopefully prevent us from trying to send bounce notifications
back to `automail@360members.com` address.

Author: kobsy

app/mailers/ep_postmaster/postmaster.rb

@@ -2,7 +2,10 @@ module EpPostmaster
   class Postmaster < ::ActionMailer::Base
 
     def bounced_email(options = {})
-      @to = MailgunPost.unfurl(options.fetch(:original_sender) { options.fetch(:reply_to) })
+      @to = Array.wrap(MailgunPost.unfurl(options.fetch(:original_sender) { options.fetch(:reply_to) }))
+      @to -= EpPostmaster.configuration.ignored_bounce_emails
+      return if @to.empty?
+
       @recipient = options.fetch(:original_recipient)
       @subject = options[:original_subject]
       @error =  options[:error]

lib/ep_postmaster/configuration.rb

@@ -1,9 +1,10 @@
 module EpPostmaster
   class Configuration
-    attr_accessor :mailgun_api_key, :mailer_sender, :mailer_deliverer
+    attr_accessor :mailgun_api_key, :mailer_sender, :mailer_deliverer, :ignored_bounce_emails
 
     def initialize
       @mailgun_api_key = ""
+      @ignored_bounce_emails = []
       self.mailer_deliverer = ->(message) { message.deliver_now }
       self.bounced_email_handler = Class.new do
         def self.handle_bounced_email!(*)

mise.toml

@@ -0,0 +1,2 @@
+[tools]
+ruby = "3.4"

test/unit/mailers/ep_postmaster/postmaster_test.rb

@@ -40,6 +40,15 @@ class PostmasterTest < ActionMailer::TestCase
       should "say something in the body of the email" do
         assert_match "unable to deliver", mail.body.encoded
       end
+
+      should "not send anything when the original sender is in a list of ignored addresses" do
+        EpPostmaster.configure { |config| config.ignored_bounce_emails = [ "sender@test.test" ] }
+        message = Postmaster.bounced_email(options).message
+        EpPostmaster.configure { |config| config.ignored_bounce_emails = [] }
+
+        assert message.is_a?(ActionMailer::Base::NullMail)
+
+      end
     end
 
   private