This repository was archived by the owner on Mar 20, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 535
Correct mime-type for JSON? #2
Comments
I mean I don't really mind, just curious since I am working on a hapi version of a GraphQL server, I don't like to blindly transform someone else work. |
@SimonDegraeve I believe that's by mistake, as the accepted content-type for the request body is express-graphql/src/parseBody.js Line 37 in 34c1441
|
Indeed, that would make sense... |
Thanks for finding this! Indeed it's a bug. application/json is intended |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I noticed the middleware returns JSON with
text/json
mime-type. Any reason why it doesn't returnapplication/json
?I know there is a debate about that for a long time in the community but I thought
application/json
became the norm.The text was updated successfully, but these errors were encountered: