search

nextcloud / files_antivirus

👾 Antivirus app for Nextcloud Files
https://apps.nextcloud.com/apps/files_antivirus
GNU Affero General Public License v3.0
84 stars 35 forks source link

File size limit not respected #175

Closed wiswedel closed 3 years ago

wiswedel commented 3 years ago

Steps to reproduce

  1. Have files_antivirus configured with ClamAV as Daemon (Socket)
  2. Configure File size limit to 20000000 bytes (20 MB)
  3. Upload an infected Zip-file with 33 MB

Expected behaviour

File scan gets bypassed, file gets uploaded

Actual behaviour

File gets scanned and rejected

Server configuration

rullzer commented 3 years ago

Yes the filesize is for background scans. Since we don't know the filesize as it is being uploaded there we just scan the individual parts. (which are chunks of 10MB)

wiswedel commented 3 years ago

the filesize is for background scans

What's the scenario for files to be scanned in background? When they've been modified at rest?

wiswedel commented 3 years ago

the filesize is for background scans

What's the scenario for files to be scanned in background? When they've been modified at rest?

@rullzer Could you please respond by the end of this week? I owe a customer an answer by Monday. Thx.

rullzer commented 3 years ago

the filesize is for background scans

What's the scenario for files to be scanned in background? When they've been modified at rest?

We just scan files in the background every 4 weeks if I'm not mistaken. So that when virus definitions get updated we also scan the files again.