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.

Comments

moshe weitzman’s picture

Status: Active » Needs work

Thanks for your work ... I would really appreciate if you could break this up into smaller patches, and isolate bug fixes from features.

tmalo’s picture

Title: New features and bugs fixed » Just the bug fix
StatusFileSize
new1.28 KB

Here is a quick patch to correct the mailbox security check problem.

moshe weitzman’s picture

Status: Needs work » Fixed

fixed in 4.7/HEAD. thanks.

moshe weitzman’s picture

Title: Just the bug fix » misc features
Status: Fixed » Active

i would love to see a patch for these new features against HEAD.

cor3huis’s picture

Status: Active » Closed (won't fix)

Alhough 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.

cor3huis’s picture

Assigned: tmalo » Unassigned
Status: Closed (won't fix) » Needs review

Set 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.

ilo’s picture

Version: 4.6.x-1.x-dev » master
Status: Needs review » Needs work

Actually, there are good ideas that we can plan for further releases, so I'm moving to HEAD.

danepowell’s picture

Status: Needs work » Fixed

These 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.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.