Skip to content
This repository has been archived by the owner on Dec 8, 2023. It is now read-only.

Tidy up the JSON request body handling #21

Open
rknLA opened this issue Sep 9, 2015 · 0 comments
Open

Tidy up the JSON request body handling #21

rknLA opened this issue Sep 9, 2015 · 0 comments

Comments

@rknLA
Copy link
Contributor

rknLA commented Sep 9, 2015

#20 was a bit haphazard, and might require a bit more tidying and thought.

As mentioned in the post-mortem comments there, we need to make sure that we're checking Content-Type, and developing a reasonable strategy for error handling. My knee-jerk reaction is that if the body content isn't parsable by whichever parser we infer from Content-Type, then we should actually raise a 422 Unprocessable Entity here, instead of propagating the request any further down the pipe.

@rknLA rknLA mentioned this issue Sep 25, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant