Dont assume the post-body is json

colynb / request-ntlm

NTLM lib for nodejs

5 stars 24 forks source link

Dont assume the post-body is json #5

Open FrankyBoy opened 9 years ago

FrankyBoy commented 9 years ago

Hi! Right now the assumption is that the value provided to .post is json, which is quite bad. Two ways of fixing this: add an "if", or just let the user set the right property in the "options" object themselves.

FrankyBoy commented 9 years ago

Commit for this: https://github.com/FrankyBoy/request-ntlm/commit/b2353fba6848147a76895e5a98da93060d2ffe96 ... for some reason it seems commits dont get linked if they are pushed before the defect report ;)