From 5dafb435d8946805cbb25bccb1bfac31139915d1 Mon Sep 17 00:00:00 2001
From: Igor Kalashnikov <igor.kalashnikov@me.com>
Date: Sat, 20 Feb 2016 22:23:51 +0300
Subject: querystring: using toString for objects on querystring.escape

This commit fixes an inconsistency in querystring.escape objects handling
compared to native encodeURIComponent function.

Fixes: https://github.com/nodejs/node/issues/5309
PR-URL: https://github.com/nodejs/node/pull/5341
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
---
 lib/querystring.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

(limited to 'lib/querystring.js')

diff --git a/lib/querystring.js b/lib/querystring.js
index ad1b5861a0..bacfc4bd70 100644
--- a/lib/querystring.js
+++ b/lib/querystring.js
@@ -90,8 +90,12 @@ for (var i = 0; i < 256; ++i)
 QueryString.escape = function(str) {
   // replaces encodeURIComponent
   // http://www.ecma-international.org/ecma-262/5.1/#sec-15.1.3.4
-  if (typeof str !== 'string')
-    str += '';
+  if (typeof str !== 'string') {
+    if (typeof str === 'object')
+      str = String(str);
+    else
+      str += '';
+  }
   var out = '';
   var lastPos = 0;
 
-- 
cgit v1.2.3