search

tomas / needle

Nimble, streamable HTTP client for Node.js. With proxy, iconv, cookie, deflate & multipart support.
https://www.npmjs.com/package/needle
MIT License
1.63k stars 236 forks source link

Add support for `application/vnd.api+json` content types #362

Closed kfitzgerald closed 3 years ago

kfitzgerald commented 3 years ago

Greetings!

It's been a while, but I ran into an API that returns something called JSON-API: https://jsonapi.org/

Apparently it's too good to use regular old content-type: applcation/json, and had to invent their own mime type. Needless to say, it's still JSON with a specialized format. Woohoo!

All sarcasm aside, this PR treats their application/vnd.api+json content mime type as json.

Unit tests supporting the case added as well. Let me know how this looks and if there's anything else you need from me!

Thanks again, -Kevin

tomas commented 3 years ago

Looks great! Thanks Kevin

kfitzgerald commented 3 years ago

Thanks again @tomas! Really appreciate it.