-
Notifications
You must be signed in to change notification settings - Fork 69
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
Add option for on-demand scan instead of daemon #41
Comments
Very nice idea to improve the memory footprint, but not sure how it will be performance wise using the |
I benchmarked the difference with a very small file. The time and cpu overhead per file is about 300x (30000%). With 17 seconds of waiting, anything done synchronously on the UI would not make for an acceptable UX with the on-demand scan. This approach would make sense for independent and processes on the backend with lower volumes.
|
Feature request:
Since the validation is an on-demand function, there is little value in using the daemon.
Calling clamav on instead on-demand follows a more coherent architecture.
This would reduce the idle memory footprint by ~250MB per instance for monolithic apps.
The text was updated successfully, but these errors were encountered: