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

422 error response not parsed correctly #47

Open
thomasbalsloev opened this issue Jan 6, 2015 · 5 comments
Open

422 error response not parsed correctly #47

thomasbalsloev opened this issue Jan 6, 2015 · 5 comments

Comments

@thomasbalsloev
Copy link

Hi Chris.

It seems the error body is not parsed correctly for 422 errors.
The 422 response format is http://dev.desk.com/API/using-the-api/#validation-codes
which the error_body method is not parsing correctly.

I only get error messages like this missing the body part:
POST https://xxx.desk.com/api/v2/cases: 422

Secondly, it would be useful to be able to access the raw response body for getting the complete info of the error.

Best regards,
Thomas Balsløv.

@chriswarren
Copy link
Owner

Thanks for letting us know - do you want to take a look at fixing it? Just fork the repo and then submit a pull request and I'll get it all published.

@thomasbalsloev
Copy link
Author

Hi Chris. Sorry for getting back so late on this.
I do not have the time at my workplace for this, at least not currently.
So if another wants to take a shot at this it would be most welcome.

Thomas.

@robins35
Copy link

Just noting that I'm having the same problem, which makes this gem essentially unusable for the time being. Does anyone know if there's a way to access the error through some kind of hack, until it gets fixed?

@chriswarren
Copy link
Owner

@colinc Do you have any thoughts on this?

@milanij
Copy link

milanij commented Sep 17, 2015

Hey Guys. I hope this finds you well. I'm just wondering where we are on this. It looks like there's an open PR, but it's just been sitting there for months.

I'm wondering if there's anything that I can do. I've looked at @robins35's code, and it looks good to me. I'm not sure about the removal of pony, and if perhaps that is causing the failing build for final commit.

Please let me know. I'd love to help and get this added to the gem. Cheers.

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

No branches or pull requests

4 participants