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

Sakai doesn't log the actual values used in sending email

    Details

    • Type: Bug
    • Status: Verified
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 12.0
    • Fix Version/s: 12.0, 19.0
    • Component/s: Kernel
    • Labels:
      None
    • 12 status:
      Resolved
    • Previous Issue Keys:
      KNL-1573
    • Test Plan:
      Hide
      1. Use the email tool to send an email. 
      2. check catalina.out log file. 
      3. in addition to the original to the standard email information there should be an entry "Message sent to:" followed by the list of account names to which the email was sent. (NB: If you are running with property testMode@org.sakaiproject.email.api.EmailService=true , meaning the email is going to the log file and not an email service then you will also see the email in the log file with a list of all the email addresses to which the email is sent. )

       

      Show
      Use the email tool to send an email.  check catalina.out log file.  in addition to the original to the standard email information there should be an entry "Message sent to:" followed by the list of account names to which the email was sent. (NB: If you are running with property testMode@org.sakaiproject.email.api.EmailService=true , meaning the email is going to the log file and not an email service then you will also see the email in the log file with a list of all the email addresses to which the email is sent. )  

      Description

      When emails are logged after sending the actual addresses aren't logged but the values that the tool requested are. While it can be useful to see what the tool attempted to use I think at this low level we should be logging the actual values used.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  buckett Matthew Buckett
                  Reporter:
                  buckett Matthew Buckett
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code