clamscan - npm Package Compare versions

Comparing version 0.2.0 to 0.2.1

index.js

@@ -166,5 +166,9 @@ /*!
 			if(err || stderr) {
-				if(self.settings.debug_mode)
-					console.log(err);
-				callback(err, file, null);
+				if(err.code === 1) {
+					callback(null, file, true);
+				} else {
+					if(self.settings.debug_mode)
+						console.log(err);
+					callback(err, file, null);
+				}
 			} else {
@@ -171,0 +175,0 @@ var result = stdout.trim();

package.json

 {
   "name": "clamscan",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "author": "Kyle Farris <[e-mail]> (http://chomponllc.com)",
   "description": "Use Node JS to scan files on your server with ClamAV's clamscan binary. This is especially useful for scanning uploaded files provided by un-trusted sources.",
   "main": "index.js",
-  "contributors": [],
+  "contributors": [
+    "dietervds"
+  ],
   "scripts": {
@@ -9,0 +11,0 @@ "test": "echo 'Error: no test specified' && exit 1"

README.md

@@ -7,2 +7,8 @@ ## NodeJS Clamscan Virus Scanning Utility
 
+## Changelog
+
+### 0.2.1
+
+BUG FIX: ClamAV returns an exit code 1 when it detects a virus but `exec` was interpreting that response as an error. Checking the response with type-sensitive equivalence resolves this bug.
+
 ## How to Install
@@ -178,2 +184,2 @@
 
-Got a missing feature you'd like to use? Found a bug? Go ahead and fork this repo, build the feature and issue a pull request.
+Got a missing feature you'd like to use? Found a bug? Go ahead and fork this repo, build the feature and issue a pull request.

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

19056

increased by1.69%

Lines of code

239

increased by1.7%

Number of lines in readme file

184

increased by3.95%

No dependency changes