Improved error handling

kenduck · kenduck · commit c354195d029c · 2015-10-09T01:51:40.000-04:00
diff --git a/audit-package.js b/audit-package.js
@@ -228,7 +228,7 @@ getArtifactMatches = function(pkgs, artifacts) {
 		var artifact = undefined;
 		if(j < artifacts.length) artifact = artifacts[j];
 		var matchedArtifact = undefined;
-		while(artifact != undefined && artifact.search[1] == name) {
+		while(artifact != undefined && artifact.search != undefined && artifact.search[1] == name) {
 			if(matchedArtifact == undefined) {
 				matchedArtifact = artifact;
 			}
diff --git a/audit.js b/audit.js
@@ -297,7 +297,12 @@ function resultCallback(err, pkg, details) {
 	}
 	
 	if(err) {
-		console.log(colors.bold.red("Error running audit: " + err));
+		if(err.error) {
+			console.log(colors.bold.red("Error running audit: " + err.error + " (" + err.code + ")"));
+		}
+		else {
+			console.log(colors.bold.red("Error running audit: " + err));
+		}
 		if(err.stack) {
 			console.log(err.stack);
 		}
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "auditjs",
-  "version": "0.0.17",
+  "version": "0.0.19",
   "description": "Audit dependencies to identify known vulnerabilities and maintenance problems",
   "main": "audit-package.js",
   "bin": "./audit.js",

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ getArtifactMatches = function(pkgs, artifacts) {`
`228`	`228`	`var artifact = undefined;`
`229`	`229`	`if(j < artifacts.length) artifact = artifacts[j];`
`230`	`230`	`var matchedArtifact = undefined;`
`231`		`- while(artifact != undefined && artifact.search[1] == name) {`
	`231`	`+ while(artifact != undefined && artifact.search != undefined && artifact.search[1] == name) {`
`232`	`232`	`if(matchedArtifact == undefined) {`
`233`	`233`	`matchedArtifact = artifact;`
`234`	`234`	`}`
Original file line number	Diff line number	Diff line change
`@@ -297,7 +297,12 @@ function resultCallback(err, pkg, details) {`
`297`	`297`	`}`
`298`	`298`
`299`	`299`	`if(err) {`
`300`		`- console.log(colors.bold.red("Error running audit: " + err));`
	`300`	`+ if(err.error) {`
	`301`	`+ console.log(colors.bold.red("Error running audit: " + err.error + " (" + err.code + ")"));`
	`302`	`+ }`
	`303`	`+ else {`
	`304`	`+ console.log(colors.bold.red("Error running audit: " + err));`
	`305`	`+ }`
`301`	`306`	`if(err.stack) {`
`302`	`307`	`console.log(err.stack);`
`303`	`308`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "auditjs",`
`3`		`- "version": "0.0.17",`
	`3`	`+ "version": "0.0.19",`
`4`	`4`	`"description": "Audit dependencies to identify known vulnerabilities and maintenance problems",`
`5`	`5`	`"main": "audit-package.js",`
`6`	`6`	`"bin": "./audit.js",`