fix(ovh): disable deletion of 0 or empty string params

Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
ovh · Aug 8, 2016 · e214e84 · e214e84
1 parent 835a178
commit e214e84
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/lib/ovh.es5.js b/lib/ovh.es5.js
@@ -150,7 +150,7 @@ var Ovh = function () {
         return this.addApi(apiPath, api, apis);
       }
 
-      if (!apis[path]) {
+      if (apis[path] == null) {
         apis[path] = { _path: apis._path + '/' + path };
       }
 
@@ -279,7 +279,7 @@ var Ovh = function () {
     value: function request(httpMethod, path, params, callback, refer) {
       var _this2 = this;
 
-      if (!callback) {
+      if (callback == null) {
         callback = params;
       }
 
@@ -345,7 +345,7 @@ var Ovh = function () {
 
       // Remove undefined values
       for (var k in params) {
-        if (params.hasOwnProperty(k) && !params[k]) {
+        if (params.hasOwnProperty(k) && params[k] == null) {
           delete params[k];
         }
       }
@@ -424,7 +424,7 @@ var Ovh = function () {
       if (typeof this.timeout === 'number') {
         req.on('socket', function (socket) {
           socket.setTimeout(_this2.timeout);
-          if (!socket._events.timeout) {
+          if (socket._events.timeout != null) {
             socket.on('timeout', function () {
               return req.abort();
             });

diff --git a/lib/ovh.es6.js b/lib/ovh.es6.js
@@ -136,7 +136,7 @@ class Ovh {
       return this.addApi(apiPath, api, apis);
     }
 
-    if (!apis[path]) {
+    if (apis[path] == null) {
       apis[path] = { _path: apis._path + '/' + path };
     }
 
@@ -271,7 +271,7 @@ class Ovh {
    * @param {Object} refer: The parent proxied object
    */
   request(httpMethod, path, params, callback, refer) {
-    if (!callback) {
+    if (callback == null) {
       callback = params;
     }
 
@@ -337,7 +337,7 @@ class Ovh {
 
     // Remove undefined values
     for (let k in params) {
-      if (params.hasOwnProperty(k) && !params[k]) {
+      if (params.hasOwnProperty(k) && params[k] == null) {
         delete params[k];
       }
     }
@@ -428,7 +428,7 @@ class Ovh {
     if (typeof(this.timeout) === 'number') {
       req.on('socket', (socket) => {
         socket.setTimeout(this.timeout);
-        if (!socket._events.timeout) {
+        if (socket._events.timeout != null) {
           socket.on('timeout', () => req.abort());
         }
       });