Loading...

XML

Word

Printable

Affected Branches:
MOODLE_310_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE
Fixed Branches:
MOODLE_311_STABLE
Pull from Repository:
https://github.com/timhunt/moodle.git
Pull Main Branch:
~~MDL-70269~~
Pull Main Diff URL:
https://github.com/timhunt/moodle/compare/master...MDL-70269
Testing Instructions:
Hide

Go to "Site administration > Plugins > Antivirus plugins > Manage antivirus plugins".

Access to "Settings" for "ClamAV antivirus".

Verify that the default for "On ClamAV failure" is shown as "Refuse upload, try again".
Show
Go to "Site administration > Plugins > Antivirus plugins > Manage antivirus plugins". Access to "Settings" for "ClamAV antivirus". Verify that the default for "On ClamAV failure" is shown as "Refuse upload, try again".

~~MDL-67545~~ introduced a nice option for what to do if trying to scan a file with Clam AV fails.

This is so much better than all the other options that I think it should be default.

The three options are:

Treat files as OK - this is risky. You might end up distributing infected files.
Treat files like viruses - this scares users. It tells them they have a virus when in fact it was just our server having a hiccup.
Refuse upload, try again - this should surely be the default for new sites?

Well, it seems obvious to me, so I am going to make a patch.

has a non-specific relationship to

MDL-67545 New config option for ClamAV scanning errors to prompt user to try again later

Participants:: CiBoT, Dan Marsden, Gladys Basiana, Ruslan Kabalin, Sara Arjona (@sarjona), Tim Hunt

Estimated:

Not Specified

Remaining:

Logged:

25m