drewblas/aws-ses — pull request #30 by mdesjardins (2012-01-11 17:01:30+00:00)
Title: Added code to check for existence of html or text body when attachments ...

Hello,

This is only my second pull request, so I apologize if I'm not following proper etiquette!

I opened Issue #29 two days ago - the problem is that SES gets angry when you try to send an e-mail with an attachment and no other body text. Rather than do something doofy like create a default text body, I've tweaked the code to throw an ArgumentError with more descriptive text when someone tries to do this.

The code doesn't depend on ActionMailer, and there's an accompanying test. Let me know if you have any questions!

BTW - very nice work on this gem. It's been a lifesaver!

- Mike