[
http://jira.magnolia-cms.com/browse/MAGNOLIA-2420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=20528#action_20528
]
Alejandro Scandroli commented on MAGNOLIA-2420:
-----------------------------------------------
I just realized that my patch only works on port 465 (SSL) and doesn't support
the other gmail port 587 (TLS)
I've mixed up the configuration parameters and they are mutually exclusive.
The code should be something like this:
{code:java}
if (Boolean.valueOf((String)
this.mailParameters.get(SMTP_SSL)).booleanValue()) {
props.put("mail.smtp.socketFactory.class",
"javax.net.ssl.SSLSocketFactory");
props.put("mail.smtp.socketFactory.fallback", "false");
}
if (Boolean.valueOf((String)
this.mailParameters.get(SMTP_TLS)).booleanValue()) {
props.put("mail.smtp.starttls.enable", "true");
}
{code}
Tomorrow, I will create a new patch (this time against trunk) to fix this.
> Allow mail.smtp.starttls.enable property configuration
> ------------------------------------------------------
>
> Key: MAGNOLIA-2420
> URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-2420
> Project: Magnolia
> Issue Type: Improvement
> Components: mail
> Reporter: Marco Collovati
> Assignee: Teresa Miyar
> Priority: Minor
> Fix For: 3.6.x, 4.0
>
> Attachments: MgnlMailFactory.patch, module-mail_MAGNOLIA-2420.patch
>
>
> For some smtp server (gmail) we need to set mail.smtp.starttls.enable
> property to true when creating a mail Session
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
