[ https://jira.codehaus.org/browse/MCHANGES-303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=343062#comment-343062 ]
Dennis Lundberg commented on MCHANGES-303: ------------------------------------------ Hello Benoit, Thank you for your patch. I've had a look at it and it looks good, but I have a question. In the ProjectJavamailMailSender.java file you set the {{startTls}} property if TLS is enabled. Shouldn't that code be inside the try/catch just above it? I.e. inside the {{if( isSslMode() )}} statement? I imagine that SSL is required to be able to use TLS. > Add an option to enable tls > --------------------------- > > Key: MCHANGES-303 > URL: https://jira.codehaus.org/browse/MCHANGES-303 > Project: Maven Changes Plugin > Issue Type: Improvement > Components: announcement > Affects Versions: 2.8 > Reporter: Benoit Guerin > Attachments: enableTls.patch > > > Add an option to set ProjectJavamailMailSender.tlsEnabled to true, to allow > to use as an example GMail to send announcement emails -- This message was sent by Atlassian JIRA (v6.1.6#6162)