Details
-
Bug
-
Status: Reopened
-
Minor
-
Resolution: Unresolved
-
2.3
-
None
-
MOODLE_23_STABLE
-
MDL-34732-master
Description
While reviewing how VERP (http://en.wikipedia.org/wiki/Variable_envelope_return_path)is implemented in Moodle (see $CFG->handlebounces and related setting in config-dist.php), I realized some interesting facts:
- It cannot work in recent Moodle version (since CLI_SCRIPT constant has been introduced), yet nobody spotted it. Apparently it is not used much.
- Also, the SMTP setup instructions are obsolete as today, we cannot execute Moodle scripts via an user account that does not have write access to moodledata (even if they do not use moodledata and DB). Which itself is a good feature/check, not a bug.
- Currently it has certain design limitations. The intention (see commit bb64b51a from 2005) was that beside the email bouncing handling, activity modules could implement a callback to handle incoming emails. Not only we have much more plugin types today, but also the implementation expects that all module ids in the
{modules}
table are lower than 256 (see how pack() is used). Which can potentially lead to serious problems.
- A question comes to my mind - do we really these callbacks into plugins? I am not aware of any single plugin that would use it. And today, with the web services framework, I believe there are more better and secure way of doing such things. I would personally be happy with the proper email bouncing protection implemented.
Attachments
Issue Links
- has a non-specific relationship to
-
MDL-73864 Improve the bounce handling interface and performance
-
- Open
-
- has been marked as being related by
-
MDL-3714 Forum: Reply by e-mail
-
- Closed
-
-
MDL-31560 New message_send API is incapable of sending notfications from the $CFG->noreplyaddress.
-
- Closed
-
- is blocked by
-
MDL-25995 Email activated setting missing from user profile
-
- Closed
-
-
MDL-42868 emailstop option non functionnal
-
- Closed
-
- is duplicated by
-
MDL-3690 prompt user to change email
-
- Closed
-
-
MDL-53266 Add an inbound message handler for bounced emails and increment users bounce count
-
- Closed
-