search

modoboa / modoboa-installer

An installer for Modoboa
MIT License
264 stars 138 forks source link

Update master.cf.tpl - add missing headers #530

Closed gsloop closed 7 months ago

gsloop commented 1 year ago

Add missing headers for mail from sasl auth'd users and mynetworks.

Description of the issue/feature this PR addresses: In our environment we've got quite a number of scanners/MFC printers (and some other devices) that send mail but don't supply normal RFC required headers - namely date. Tweaking the master.cf fixes this in a way that I don't think would adversely impact anyone.

Current behavior before PR: Missing headers, like date would cause the file to show up in quarantined list for amavis (though I don't think any in our case actually got quarantined - though I'm not positive.)

Desired behavior after PR is merged: This simply adds missing header fields required by RFC, though only for clients who have done sasl or are part of my-networks, so this should be safe from breaking DKIM etc. (And shouldn't allow abuse from non-priv'd clients/senders.)

tonioo commented 7 months ago

@gsloop Thanks for the contribution