Page MenuHomePhabricator

Improve Amazon SES code error handling behavior
ClosedPublic

Authored by epriestley on Apr 6 2016, 12:02 AM.
Tags
None
Referenced Files
F18776440: D15632.diff
Fri, Oct 10, 10:03 PM
F18671525: D15632.id37670.diff
Thu, Sep 25, 4:10 AM
F18642909: D15632.diff
Fri, Sep 19, 1:10 AM
F18588143: D15632.id37671.diff
Sep 11 2025, 11:23 PM
F18586025: D15632.id37671.diff
Sep 11 2025, 5:12 PM
F18583420: D15632.id37670.diff
Sep 11 2025, 10:20 AM
F18356491: D15632.diff
Aug 27 2025, 5:47 AM
F17868754: D15632.diff
Jul 28 2025, 9:26 AM
Subscribers
None

Details

Summary

Fixes T10728. Fixes T10476. SES uses third-party code with unique, creative ideas about error handling.

  • Make the error handling behavior more correct, so it doesn't try to use undefined variables.
  • Simplify the error handling behavior (throw exceptions sooner, remove redundant code).
  • Explicitly test for -smtp misconfigurations. These can arise if you read the wrong column out of the table in the AWS docs, as in T10728.
  • Explicitly test for SimpleXML, to catch T10476 before it does damage.
Test Plan
  • Configured SES to use a bogus SMTP endpoint.
  • Faked past the SMTP check, hit sane error on the connection.
  • Undid faking, hit immediate hard stop on the STMP check.

Diff Detail

Repository
rP Phabricator
Branch
ses2
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 11536
Build 14404: Run Core Tests
Build 14403: arc lint + arc unit