Page MenuHomePhabricator

Improve Amazon SES code error handling behavior
ClosedPublic

Authored by epriestley on Apr 6 2016, 12:02 AM.
Tags
None
Referenced Files
F16651337: D15632.diff
Tue, Jun 17, 11:37 AM
F16609840: D15632.diff
Tue, Jun 17, 7:23 AM
Unknown Object (File)
May 6 2025, 8:48 AM
Unknown Object (File)
Apr 24 2025, 5:46 AM
Unknown Object (File)
Apr 17 2025, 8:29 PM
Unknown Object (File)
Mar 31 2025, 11:33 PM
Unknown Object (File)
Mar 28 2025, 12:25 PM
Unknown Object (File)
Mar 27 2025, 7:00 PM
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