Skip to content

Encode request data as JSON when Content-Type is set to application/json#238

Open
danhardman wants to merge 1 commit intoded:masterfrom
danhardman:master
Open

Encode request data as JSON when Content-Type is set to application/json#238
danhardman wants to merge 1 commit intoded:masterfrom
danhardman:master

Conversation

@danhardman
Copy link

The issue has been open for a while with a bunch of +1s so I thought I'd give it a go.

This is untested as npm test wasn't able to run, something to do with spawning phantomjs so if anyone could give me a hand figuring out how to test this properly that'd be great. However, it's a pretty simple change so everything should be good.

@AWare
Copy link

AWare commented Nov 18, 2016

I got caught out by this issue today, this pr would be really helpful.

@danhardman
Copy link
Author

@AWare This repo is dead, I recommend finding an alternative!

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

Successfully merging this pull request may close these issues.

2 participants