Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-31050

Added default (dummy) email sender address if 'mail.support' is not set.

    XMLWordPrintable

    Details

    • 11 status:
      Resolved
    • Test Plan:
      Hide

      check that there is NO property mail.support in the properties file.

      add non-registered user to a site and opt not to notify them about the site's availability, check that the "welcome to sakai" email is sent (either in logs if email isnt enabled, or by an email actually being sent). NB this is not the "site availability notification email" which is a totally different thing.

      As a user with a username that is an email address, use Account to change the email address, then check that an email is sent (either in logs if email isnt enabled, or by an email actually being sent)

      Show
      check that there is NO property mail.support in the properties file. add non-registered user to a site and opt not to notify them about the site's availability, check that the "welcome to sakai" email is sent (either in logs if email isnt enabled, or by an email actually being sent). NB this is not the "site availability notification email" which is a totally different thing. As a user with a username that is an email address, use Account to change the email address, then check that an email is sent (either in logs if email isnt enabled, or by an email actually being sent)

      Description

      if mail.support is not set in the sakai properties file then the New Account and Account validation messages are not sent. So, in the absence of mail.support we construct a dummy address "support@<servername>" so the emails are sent (albeit with a reply-to address that doesnt work).

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  ouit0408 Neelam Bhandari
                  Reporter:
                  adamm Adam Marshall
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration