[
http://jira.jboss.com/jira/browse/JBPM-848?page=comments#action_12379543 ]
Tom Baeyens commented on JBPM-848:
----------------------------------
Fady, i don't get your remark.
As far as I remember, it should be possible for people to specify the property
jbpm.mail.from.address in their jbpm.cfg.xml file. Can you try to reproduce and compare
the problem report of Sebastian with the docs and the code ?
In case our code is the problem, how do you think we should fix it ?
Incorrect mail.smtp.host property + configurable mail from address
------------------------------------------------------------------
Key: JBPM-848
URL:
http://jira.jboss.com/jira/browse/JBPM-848
Project: JBoss jBPM
Issue Type: Patch
Components: Core Engine
Affects Versions: jBPM jPDL 3.2 beta 2
Reporter: Bruno Dumon
Assigned To: Tom Baeyens
Fix For: jBPM jPDL 3.2.2
Attachments: mailpatch_20070222.txt, mailpatch_v2.txt
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/M...
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.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira