Merge pull request #1 from carlos8f/json_parser_2

Add test and avoid parse exception

Author: mikefrey

lib/json_parser.js

@@ -23,10 +23,12 @@ JSONParser.prototype.write = function(buffer) {
 }
 
 JSONParser.prototype.end = function() {
-  var fields = JSON.parse(this.data.toString('utf8'))
-  for (var field in fields) {
-    this.onField(field, fields[field]);
-  }
+  try {
+    var fields = JSON.parse(this.data.toString('utf8'))
+    for (var field in fields) {
+      this.onField(field, fields[field]);
+    }
+  } catch (e) {}
   this.data = null;
 
   this.onEnd();

test/integration/test-json.js

@@ -0,0 +1,38 @@
+var common = require('../common');
+var formidable = common.formidable;
+var http = require('http');
+var assert = require('assert');
+
+var testData = {
+  numbers: [1, 2, 3, 4, 5],
+  nested: { key: 'value' }
+};
+
+var server = http.createServer(function(req, res) {
+    var form = new formidable.IncomingForm();
+
+    form.parse(req, function(err, fields, files) {
+        assert.deepEqual(fields, testData);
+
+        res.end();
+        server.close();
+    });
+});
+
+var port = common.port;
+
+server.listen(port, function(err){
+    assert.equal(err, null);
+
+    var request = http.request({
+        port: port,
+        method: 'POST',
+        headers: {
+            'Content-Type': 'application/json'
+        }
+    });
+
+    request.write(JSON.stringify(testData));
+    request.end();
+});
+