fix: getOptions change to clone

cojs · Mar 24, 2017 · 39d2e74 · 39d2e74
1 parent eb10905
commit 39d2e74
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 12 deletions.
diff --git a/lib/form.js b/lib/form.js
@@ -22,7 +22,7 @@ var utils = require('./utils');
 
 module.exports = function(req, opts){
   req = req.req || req;
-  opts = utils.getOptions(opts);
+  opts = utils.clone(opts);
   var queryString = opts.queryString || {};
 
   // keep compatibility with qs@4

diff --git a/lib/json.js b/lib/json.js
@@ -25,7 +25,7 @@ var strictJSONReg = /^[\x20\x09\x0a\x0d]*(\[|\{)/;
 
 module.exports = function(req, opts){
   req = req.req || req;
-  opts = utils.getOptions(opts);
+  opts = utils.clone(opts);
 
   // defaults
   var len = req.headers['content-length'];

diff --git a/lib/text.js b/lib/text.js
@@ -20,7 +20,7 @@ var utils = require('./utils');
 
 module.exports = function(req, opts){
   req = req.req || req;
-  opts = utils.getOptions(opts);
+  opts = utils.clone(opts);
 
   // defaults
   var len = req.headers['content-length'];

diff --git a/lib/utils.js b/lib/utils.js
@@ -3,14 +3,11 @@
  * Module dependencies.
  */
 
-exports.getOptions = function (opts) {
+exports.clone = function (opts) {
+  var options = {};
   opts = opts || {};
-  return {
-    limit: opts.limit,
-    strict: opts.strict,
-    queryString: opts.queryString,
-    qs: opts.qs,
-    returnRawBody: opts.returnRawBody,
-    encoding: opts.encoding,
-  };
+  for (var key in opts) {
+    options[key] = opts[key];
+  }
+  return options;
 }