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

Error responses should contain more information #5

Closed
rikkit opened this issue Sep 1, 2013 · 3 comments
Closed

Error responses should contain more information #5

rikkit opened this issue Sep 1, 2013 · 3 comments

Comments

@rikkit
Copy link
Member

rikkit commented Sep 1, 2013

Commit 1729119 added a switch to catch request exceptions from api actions, should preserve more information if this is used.

See this StackOverflow comment

@rikkit
Copy link
Member Author

rikkit commented Mar 26, 2015

Expanding this issue to cover improving the error codes. "Bad Method Signature" isn't descriptive and doesn't suggest a solution - in this case it'd be to make sure your api keys are correct

error

@rikkit rikkit changed the title Error response should contain more information Error responses should contain more information Mar 26, 2015
@rikkit
Copy link
Member Author

rikkit commented Apr 4, 2015

API returns "Missing Parameters" error code for albums which don't exist

{
  "error": 6,
  "message": "Album not found",
  "links": []
}

Code 6 corresponds to LastResponseStatus.MissingParameters, which implies there is something wrong with the request itself.

@rikkit rikkit added this to the v0.3.0 milestone Jun 11, 2015
@rikkit
Copy link
Member Author

rikkit commented Aug 22, 2017

Tidying issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant