Paths

Table of Contentst

Improve Amazon SES code error handling behavior further
Needs ReviewPublic
Actions

Authored by avivey on Jul 27 2017, 7:34 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 18, 7:39 AM

	Unknown Object (File)
	Wed, Dec 11, 7:26 AM

	Unknown Object (File)
	Wed, Dec 11, 12:58 AM

	Unknown Object (File)
	Sat, Dec 7, 1:36 AM

	Unknown Object (File)
	Fri, Dec 6, 10:41 AM

	Unknown Object (File)
	Tue, Dec 3, 2:14 PM

	Unknown Object (File)
	Fri, Nov 29, 12:40 PM

	Unknown Object (File)
	Fri, Nov 29, 12:11 AM

View All 99 Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley
chad
amckinley

Group Reviewers

Blessed Reviewers

Summary

If SES gives an error message, pull it into the exception.

Test Plan

Send computer back in time, ./bin/worker execute, see useful message.

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 17812
Build 23922: Run Core Tests
Build 23921: arc lint + arc unit

Event Timeline

avivey created this revision.Jul 27 2017, 7:34 AM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 27 2017, 7:34 AM

Herald added a subscriber: epriestley. · View Herald Transcript

Harbormaster completed remote builds in B17812: Diff 43970.Jul 27 2017, 7:35 AM

avivey added inline comments.Jul 27 2017, 8:51 AM

externals/amazon-ses/ses.php
520–522	Can this blow up if the XML isn't built correctly?

avivey added reviewers: epriestley, chad, amckinley.Jul 27 2017, 9:22 PM

amckinley resigned from this revision.Apr 24 2018, 4:40 PM

Herald added a subscriber: Korvin. · View Herald TranscriptApr 24 2018, 4:40 PM

Revision Contents
Changeset List

Path

Size

externals/

amazon-ses/

ses.php

43 lines

Diff 43970

View Options

externals/amazon-ses/ses.php

Improve Amazon SES code error handling behavior furtherNeeds ReviewPublicActions