nov/fb_graph — pull request #173 by timblinq (2012-01-19 20:56:13+00:00)
Title: Addition of exceptions for facebook ad api failures

Hello,

We've added a bunch of exceptions to handle the case of getting ad api failures back from Facebook that are not necessarily related to validation issues. In our testing and experience on production these exceptions come back from Facebook as errors on their end, and submitting the same request again a second (or third) time will eventually work. Because of this these new exceptions are using the HTTP status code 500 instead of 400 for a BadRequest.

Also added are tests to cover these new exceptions and the entire test suite passes.

Let me know if this doesn't make sense or if we need to approach this a different way to fit in with your convention for the project.

Thanks!

- Tim Gourley and Zack Adams