Enable non-object JSON bodies

.gitignore

@@ -1 +1,2 @@
 node_modules
+.idea

request.js

@@ -1482,7 +1482,7 @@ Request.prototype.json = function (val) {
 
   self._json = true
   if (typeof val === 'boolean') {
-    if (typeof self.body === 'object') {
+    if (self.body !== undefined) {
       self.body = safeStringify(self.body)
       if (!self.hasHeader('content-type')) {
         self.setHeader('content-type', 'application/json')

tests/server.js

@@ -77,6 +77,24 @@ exports.createPostValidator = function (text, reqContentType) {
   }
   return l
 }
+exports.createPostJSONValidator = function (value, reqContentType) {
+  var l = function (req, resp) {
+    var r = ''
+    req.on('data', function (chunk) {r += chunk})
+    req.on('end', function () {
+      var parsedValue = JSON.parse(r)
+      assert.deepEqual(parsedValue, value)
+      if (reqContentType) {
+        assert.ok(req.headers['content-type'])
+        assert.ok(~req.headers['content-type'].indexOf(reqContentType))
+      }
+      resp.writeHead(200, {'content-type':'application/json'})
+      resp.write(r)
+      resp.end()
+    })
+  }
+  return l
+}
 exports.createGetResponse = function (text, contentType) {
   var l = function (req, resp) {
     contentType = contentType || 'text/plain'

tests/test-json-request.js

@@ -0,0 +1,54 @@
+'use strict'
+
+var server = require('./server')
+  , stream = require('stream')
+  , request = require('../index')
+  , tape = require('tape')
+
+var s = server.createServer()
+
+tape('setup', function(t) {
+  s.listen(s.port, function() {
+    t.end()
+  })
+})
+
+function testJSONValue(testId, value) {
+  tape('test ' + testId, function(t) {
+    var testUrl = '/' + testId
+    s.on(testUrl, server.createPostJSONValidator(value, 'application/json'))
+    var opts = {
+      method: 'PUT',
+      uri: s.url + testUrl,
+      json: true,
+      body: value
+    }
+    request(opts, function (err, resp, body) {
+      t.equal(err, null)
+      t.equal(resp.statusCode, 200)
+      t.deepEqual(body, value)
+      t.end()
+    })
+  })
+}
+
+testJSONValue('jsonNull', null)
+testJSONValue('jsonTrue', true)
+testJSONValue('jsonFalse', false)
+testJSONValue('jsonNumber', -289365.2938)
+testJSONValue('jsonString', 'some string')
+testJSONValue('jsonArray', ['value1', 2, null, 8925.53289, true, false, ['array'], { object: 'property' }])
+testJSONValue('jsonObject', {
+  trueProperty: true,
+  falseProperty: false,
+  numberProperty: -98346.34698,
+  stringProperty: 'string',
+  nullProperty: null,
+  arrayProperty: ['array'],
+  objectProperty: { object: 'property' }
+})
+
+tape('cleanup', function(t) {
+  s.close()
+  t.end()
+})