Uploaded image for project: 'jBPM'
  1. jBPM
  2. JBPM-848

Incorrect mail.smtp.host property + configurable mail from address

    Details

      Description

      jBPM incorrectly sets "jbpm.mail.smtp.host" instead of "mail.smtp.host" in the properties passed to JavaMail. As a consequence, JavaMail always tries to send to localhost. It seems this was introduced by accident here:
      http://fisheye.jboss.com/browse/JBPM/jbpm.3/src/java.jbpm/org/jbpm/mail/Mail.java?r1=1.4&r2=1.5

      The attached patch fixes this.

      Also, the from address for mails is currently hardcoded to "jbpm@noreply". Many SMTP servers don't allow an invalid origin domain name (as do many spam filters), this is the exception I get:

      javax.mail.SendFailedException: Sending failed;
      nested exception is:
      class javax.mail.SendFailedException: Invalid Addresses;
      nested exception is:
      class javax.mail.SendFailedException: 450 <jbpm@noreply>: Sender address rejected: Domain not found

      The attached patch also makes the from address configurable via jbpm.cfg.xml.

      Patches are against current CVS and taken with the jbpm.3/jpdl directory as root.

      Thanks for taking the time to look at this.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                tom.baeyens Tom Baeyens
                Reporter:
                bdumon Bruno Dumon
              • Votes:
                1 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: