This patch provided the following :
- corrected bug preventing correct checking of security parameter of the mailbox
* added email informing user that the submission has been processed.
* prevent mailhandler to send email to strangers (potential spammers).
* command matching now with preg_match, something like "truc: bidule" is now allowed.
* added Mandatory Commands : commands that will override those provided in the email.
* added optional AntiSpam code : this code is checked instead of the password. Thus providing protection from spammer/viruses without making password wandering on the Internet
* 'post by mail' permission is now per mailbox
* 'mailhandler settings' permission allow the user to change his settings
in order to this patch to work you will need to add an extra field to the mailhandler table, like this :
ALTER TABLE `mailhandler` ADD `mandatory_commands` TEXT AFTER `commands` ;
Note : this patch is related to 4.6 version of the module but include the javanaut and weitzman modifications.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | mailhandler.module.bugfix.patch | 1.28 KB | tmalo |
| mailhandler.module.tmalo.patch | 26.39 KB | tmalo |
Comments
Comment #1
moshe weitzman commentedThanks for your work ... I would really appreciate if you could break this up into smaller patches, and isolate bug fixes from features.
Comment #2
tmalo commentedHere is a quick patch to correct the mailbox security check problem.
Comment #3
moshe weitzman commentedfixed in 4.7/HEAD. thanks.
Comment #4
moshe weitzman commentedi would love to see a patch for these new features against HEAD.
Comment #5
cor3huis commentedAlhough status set to won't fix, this is the won't fix for the 4.6.x version these fixes are already in latest current Mailhandler. Advised to upgrade your mailhandler to latest version.
Comment #6
cor3huis commentedSet to review realized there was a bigger patch with good idea that might be beneficial for the current version. Need to see what is already in place and what is not.
Comment #7
ilo commentedActually, there are good ideas that we can plan for further releases, so I'm moving to HEAD.
Comment #8
danepowell commentedThese should all be resubmitted as individual feature requests against the appropriate versions if there is still interest in them- most have already been implemented or are of marginal general interest.