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

Update spamhaus.sh #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ShamimIslam
Copy link

I have changed this to accept a few flags to make it runnable from cron quietly, runnable from rc.local with cached content, cached spamhous drop.lasso, and have allowed for a dry run of the downloaded file.

The previous version would automatically make the mail server unprotected should there be an error in downloading the spamhaus file.

The downloaded file can be further validated before it is moved but at this time, this is good enough. If the spamhaus file is empty, we ignore it. It should probably generate an email but I have not gotten to that yet.

I have changed this to accept a few flags to make it runnable from cron quietly, runnable from rc.local with cached content, cached spamhous drop.lasso, and have allowed for a dry run of the downloaded file.

The previous version would automatically make the mail server unprotected should there be an error in downloading the spamhaus file.

The downloaded file can be further validated before it is moved but at this time, this is good enough. If the spamhaus file is empty, we ignore it. It should probably generate an email but I have not gotten to that yet.
@ShamimIslam
Copy link
Author

I used your script for a while and found I needed to make these changes in order to run it on an unmonitored server so that I could be assured the mail server would stay protected and that I would not get a lot of unnecessary cron mail.

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

Successfully merging this pull request may close these issues.

1 participant