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

Upgrade MailScanner #50

Open
JohnMertz opened this issue Nov 4, 2024 · 1 comment
Open

Upgrade MailScanner #50

JohnMertz opened this issue Nov 4, 2024 · 1 comment

Comments

@JohnMertz
Copy link
Collaborator

There are fairly light, but essential modifications made to the version of MailScanner shipped with the Jessie release.

It is necessary that we ship an updated version of MailScanner with Next, which means porting our modifications. I will start by simply porting the patches directly to a fork, but will then see if we can accomplish what we need with a CustomConfig module only. See MailWatch as an example of how to integrate a custom module.

As long as we can still write to the SpamCheck header, we should not need to modify the MailScanner code directly. If we can do so with a module, this should allow us to simply use the upstream version of MailScanner with a custom config only.

@JohnMertz
Copy link
Collaborator Author

I've gone through and re-applied all of the changes that we have in /opt/MailScanner on our up-to-date appliance to the files as they existed upstream in the version used in Current:

MailCleaner/v4@f6f8698

I'm now working on evaluating which patches need to be applied to the latest v5 version and which could be migrated to a custom module.

For lack of a better place to put this, note that SweepViruses.pm does not need to be updated since it already has support of ESET and adds upstream support to not retry on batch failure with the option Retry Batch Scanner = no. Note that ClamAV support is deprecated in favor of just using the existing clamd socket. I need to implement this in our config file.

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

No branches or pull requests

1 participant