Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exposing additional information from an error response #130

Merged
merged 1 commit into from
Apr 2, 2015

Conversation

gerardo-orozco
Copy link
Contributor

I've experienced that in some edge cases, facebook simply says Invalid parameter in the error message property. This is not useful at all in facepy's exceptions as it doesn't really say anything and the real/useful error information is in other properties.

This pull request simply exposes such error information as exception attributes so that anyone who needs them have them handy and leaves the current representation untouched.

jgorset added a commit that referenced this pull request Apr 2, 2015
Exposing additional information from an error response
@jgorset jgorset merged commit 456eb70 into jgorset:master Apr 2, 2015
@jgorset
Copy link
Owner

jgorset commented Apr 2, 2015

Awesome, thanks @gerardo-orozco! ✨ 🍰 ✨

@gerardo-orozco gerardo-orozco deleted the expose-additonal-error-info branch April 2, 2015 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants