Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scanStream returning isInfected = true even when its not #110

Closed
iUnstable0 opened this issue Feb 16, 2023 · 4 comments
Closed

scanStream returning isInfected = true even when its not #110

iUnstable0 opened this issue Feb 16, 2023 · 4 comments
Labels
documentation Suggested fixes to documentation

Comments

@iUnstable0
Copy link

[04:17:12.699] [log] node-clam: Attempting to establish socket/TCP connection for "scanStream"
[04:17:12.702] [log] node-clam: using local unix domain socket: /tmp/clamd.socket
[04:17:12.703] [log] node-clam: Received final data from stream.
[04:17:12.704] [log] node-clam: The input stream has dried up.
[04:17:12.732] [log] node-clam: Received output from ClamAV Socket.
[04:17:12.733] [log] node-clam: ClamAV is done scanning.
[04:17:12.733] [log] node-clam: Raw Response:  stream: OK 
[04:17:12.733] [log] node-clam: File is OK!
[04:17:12.733] [log] Stream is infected! Booo!
[04:17:12.733] [log] node-clam: Socket/Host connection closed.
[04:17:12.734] [log] node-clam: ClamAV socket has been closed! false
						const clamAV = clamscan.scanStream(fileStream, (err, isInfected) => {
							if (err) return console.error(err);
							if (isInfected) return console.log("Stream is infected! Booo!");
							console.log("Stream is not infected! Yay!");
							fileBuffer = Buffer.concat(chunks);
						});
@eviltik
Copy link

eviltik commented Mar 10, 2024

"isInfected" should be "clamavReport".

I.E isInfected is an object having isInfected attribute.

@kylefarris
Copy link
Owner

Is this just a recommendation to fix the documentation? Just want to be sure.

@eviltik
Copy link

eviltik commented Mar 18, 2024

yep, just a documentation fix.

@kylefarris kylefarris added the documentation Suggested fixes to documentation label Mar 18, 2024
@eviltik
Copy link

eviltik commented Mar 18, 2024

Nice & quick shot. Thank you, and ty for all the job, this lib is running smooth !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Suggested fixes to documentation
Projects
None yet
Development

No branches or pull requests

3 participants