Revert "url: significantly improve the performance of the url module"

This reverts commit 3fd7fc4. It was agreed that this change contained too much potential ecosystem breakage, particularly around the inability to `delete` properties off a `Url` object. It may be re-introduced for a later release, along with better work on ecosystem compatibility. PR-URL: #1602 Reviewed-By: Mikeal Rogers <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Forrest L Norvell <[email protected]> Reviewed-By: Chris Dickinson <[email protected]> Reviewed-By: Isaac Z. Schlueter <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]>
nodejs · May 4, 2015 · 702997c · 702997c
1 parent 0daed24
commit 702997c
Show file tree

Hide file tree

Showing 6 changed files with 529 additions and 2,786 deletions.
diff --git a/lib/_http_client.js b/lib/_http_client.js
@@ -20,7 +20,7 @@ function ClientRequest(options, cb) {
   OutgoingMessage.call(self);
 
   if (typeof options === 'string') {
-    options = url.parse(options).toJSON();
+    options = url.parse(options);
   } else {
     options = util._extend({}, options);
   }

diff --git a/lib/https.js b/lib/https.js
@@ -107,7 +107,7 @@ exports.Agent = Agent;
 
 exports.request = function(options, cb) {
   if (typeof options === 'string') {
-    options = url.parse(options).toJSON();
+    options = url.parse(options);
   } else {
     options = util._extend({}, options);
   }