aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Heinemeier Hansson <david@loudthinking.com>2018-09-18 17:13:49 -0700
committerDavid Heinemeier Hansson <david@loudthinking.com>2018-09-18 17:13:49 -0700
commit029c693f19ee088af329620fb729dd9005080755 (patch)
tree0601084c0857ebb3a0ef23df6dbd5d1b9de3e5b9
parent0f140fe15852b829d22864a2f7664440204ac723 (diff)
downloadrails-029c693f19ee088af329620fb729dd9005080755.tar.gz
rails-029c693f19ee088af329620fb729dd9005080755.tar.bz2
rails-029c693f19ee088af329620fb729dd9005080755.zip
Make a note for tying inbound email to exception
Then InboundEmail doesn't need to serialize or track the exception that went with it. The arrow will point the other way.
-rw-r--r--lib/action_mailroom/mailbox.rb2
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/action_mailroom/mailbox.rb b/lib/action_mailroom/mailbox.rb
index 0d03f4be6b..a2c097a42f 100644
--- a/lib/action_mailroom/mailbox.rb
+++ b/lib/action_mailroom/mailbox.rb
@@ -31,6 +31,8 @@ class ActionMailroom::Mailbox
inbound_email.delivered!
rescue => exception
inbound_email.failed!
+
+ # TODO: Include a reference to the inbound_email in the exception raised so error handling becomes easier
rescue_with_handler(exception) || raise
end