Allow a VERP "bounceprefix" that's not exactly 4 characters
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mahara |
Fix Released
|
Low
|
Aaron Wells |
Bug Description
Mahara has a VERP system to identify users whose emails are bouncing, and disable sending to their email address. It uses a $cfg->bounceprefix setting to generated the VERP address; this is the part of the email that comes before the VERP stuff.
Currently, it arbitrarily forces this prefix to be exactly 4 characters, but there doesn't appear to be a reason for this, except that the process_email() function contains a lot of calls to substr() with hard-coded offsets that assume the prefix is exactly 4 characters.
I want to change this so that the prefix can be any length. The reason for this, is so I can use "verp+" as the prefix, which in our mail system will forward them all to the "verp[at]
Changed in mahara: | |
status: | In Progress → Fix Committed |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
https:/ /reviews. mahara. org/2802