Details

    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      Add a reply-to header to all emails sent from Moodle. This is a copy from Debian bug - see: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=610773 .
      The latest patch from Monica is attached there: http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=20;filename=add.reply-to.patch;att=1;bug=610773 . I see no problems in implementing it but that will be a change to the way Moodle sends emails at the moment - what do you guys think about it?

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              tsala Helen Foster added a comment -

              Tomasz, thanks for your suggested improvement. Assigning to moodle.com for consideration.

              Show
              tsala Helen Foster added a comment - Tomasz, thanks for your suggested improvement. Assigning to moodle.com for consideration.
              Hide
              manoelcampos Manoel Campos da Silva Filho added a comment -

              Patch to Moodle 1.9 that add replyTo header to email (based on http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=610773)

              Show
              manoelcampos Manoel Campos da Silva Filho added a comment - Patch to Moodle 1.9 that add replyTo header to email (based on http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=610773 )
              Hide
              manoelcampos Manoel Campos da Silva Filho added a comment -

              Patch to Moodle 2.2 that add replyTo header to email (based on http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=610773)

              Show
              manoelcampos Manoel Campos da Silva Filho added a comment - Patch to Moodle 2.2 that add replyTo header to email (based on http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=610773 )
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Hi Tomasz, Manoel,

              I must admit that I'm not convinced by this patch. IMO, we should not be setting a reply-to address - this is unexpected behaviour and Google are doing the wrong thing IMO.
              Google is not intended as an SMTP service. An e-mail service should not munge the From address.

              This will likely be discussed in relation to MDL-3714 so I'm adding a link so that we consider this as part of that issue.

              This will require some careful re-reading of RFC-822 (oh yay)...

              Andrew

              Show
              dobedobedoh Andrew Nicols added a comment - Hi Tomasz, Manoel, I must admit that I'm not convinced by this patch. IMO, we should not be setting a reply-to address - this is unexpected behaviour and Google are doing the wrong thing IMO. Google is not intended as an SMTP service. An e-mail service should not munge the From address. This will likely be discussed in relation to MDL-3714 so I'm adding a link so that we consider this as part of that issue. This will require some careful re-reading of RFC-822 (oh yay)... Andrew
              Hide
              jrh18 Jason Hardin added a comment -

              is this still and issue in 3.1?

              Show
              jrh18 Jason Hardin added a comment - is this still and issue in 3.1?
              Hide
              tmuras Tomasz Muras added a comment -

              I actually don't know if it's still an issue or not. But I didn't have any problems with this reply-to issue and didn't hear any complains about it for years now. Let's close it.

              Show
              tmuras Tomasz Muras added a comment - I actually don't know if it's still an issue or not. But I didn't have any problems with this reply-to issue and didn't hear any complains about it for years now. Let's close it.
              Hide
              jrh18 Jason Hardin added a comment -

              reopen if someone actually identifies this as an issue.

              Show
              jrh18 Jason Hardin added a comment - reopen if someone actually identifies this as an issue.

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: