saucelabs - npm Package Compare versions

Comparing version 0.1.0 to 0.1.1

lib/SauceLabs.js

@@ -271,11 +271,16 @@ var crypto      = require('crypto');
       .on('end', function () {
+        var res;
+
         try {
-          if (response.statusCode === 200) {
-            callback(null, JSON.parse(result));
-          } else {
-            callback(JSON.parse(result));
-          }
-        } catch (err) {
+          res = JSON.parse(result);
+        } catch (e) {
           callback('Could not parse response: ' + result);
+          return;
         }
+
+        if (response.statusCode === 200) {
+          callback(null, res);
+        } else {
+          callback(res);
+        }
       });
@@ -282,0 +287,0 @@ });

package.json

@@ -5,3 +5,3 @@ {
   "description": "A wrapper around Sauce Labs REST API",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "homepage": "https://github.com/holidayextras/node-saucelabs",
@@ -8,0 +8,0 @@ "repository": {

README.md

@@ -39,3 +39,3 @@ # node-saucelabs [![Build Status](https://secure.travis-ci.org/holidayextras/node-saucelabs.png)](http://travis-ci.org/holidayextras/node-saucelabs)
   password: "your-sauce-api-key"
-})
+});
 
@@ -50,16 +50,18 @@ myAccount.getAccountDetails(function (err, res) {
       console.log(res);
-      myAccount.getJobs(function (err, res) {
+      myAccount.getJobs(function (err, jobs) {
         // Get a list of all your jobs
-        for (k in jobs) {
-          myAccount.showJob(jobs[k].id, function (err, res) {
-            var str = res.id + ": Status: " + res.status;
-            if (res.error) {
-              str += "\033[31m Error: " + res.error + " \033[0m";
-            }
-            console.log(str);
-          });
+        for (var k in jobs) {
+          if ( jobs.hasOwnProperty( k )) {
+            myAccount.showJob(jobs[k].id, function (err, res) {
+              var str = res.id + ": Status: " + res.status;
+              if (res.error) {
+                str += "\033[31m Error: " + res.error + " \033[0m";
+              }
+              console.log(str);
+            });
+          }
         }
-      })
-    })
-  })
+      });
+    });
+  });
 });
@@ -66,0 +68,0 @@ ```

New alerts

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

Non-existent author

Supply chain risk

The package was published by an npm account that no longer exists.

Found 1 instance in 1 package

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Number of lines in readme file

291

increased by0.69%

Number of high supply chain risk alerts

0

decreased by-100%

Worsened metrics

Total package byte prevSize

29567

decreased by-0.91%

Number of package files

14

decreased by-17.65%

Lines of code

583

decreased by-0.85%

No dependency changes