[
https://issues.apache.org/jira/browse/OFBIZ-1256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12554024
]
Chris Howe commented on OFBIZ-1256:
-----------------------------------
Looking at BJ's suggestions for the multiple email accounts, it shouldn't
affect this issue.
A mutliple account solution would make the appropriate data available in the
context.get(""); for what ever account is sending the email. As a side note to
BJ's suggestion, the account information should be stored in the database, not
in property files. The general.properties are simply default configurations to
pass to JavaMail if nothing is provided in the context.
> Send mail through gmail with the javamail api in ofbiz
> ------------------------------------------------------
>
> Key: OFBIZ-1256
> URL: https://issues.apache.org/jira/browse/OFBIZ-1256
> Project: OFBiz
> Issue Type: New Feature
> Components: content
> Affects Versions: SVN trunk
> Environment: All
> Reporter: Fabien Carrion
> Assignee: Jacques Le Roux
> Priority: Minor
> Fix For: SVN trunk
>
> Attachments: emailssl_diff.patch, gmail.patch
>
>
> This a small patch to accept ssl connections to smtp servers.
> Index: content/src/org/ofbiz/content/email/EmailServices.java
> ===================================================================
> --- content/src/org/ofbiz/content/email/EmailServices.java (revision
> 576503)
> +++ content/src/org/ofbiz/content/email/EmailServices.java (working copy)
> @@ -129,6 +129,8 @@
>
> String sendFrom = (String) context.get("sendFrom");
> String sendType = (String) context.get("sendType");
> + String port = (String) context.get("port");
> + String ssl = (String) context.get("ssl");
> String sendVia = (String) context.get("sendVia");
> String authUser = (String) context.get("authUser");
> String authPass = (String) context.get("authPass");
> @@ -151,6 +153,12 @@
> if (authUser != null && authUser.length() > 0) {
> useSmtpAuth = true;
> }
> + if (port == null || port.length() == 0) {
> + port = UtilProperties.getPropertyValue("general.properties",
> "mail.smtp.port", "25");
> + }
> + if (ssl == null || ssl.length() == 0) {
> + ssl = UtilProperties.getPropertyValue("general.properties",
> "mail.smtp.ssl", "javax.net.ssl.SSLSocketFactory");
> + }
> } else if (sendVia == null) {
> return ServiceUtil.returnError("Parameter sendVia is required
> when sendType is not mail.smtp.host");
> }
> @@ -168,6 +176,14 @@
> try {
> Properties props = System.getProperties();
> props.put(sendType, sendVia);
> + if (port != null && port.length() > 0) {
> + props.put("mail.smtp.port", port);
> + props.put("mail.smtp.socketFactory.port", port);
> + }
> + if (ssl != null && ssl.length() > 0) {
> + props.put("mail.smtp.socketFactory.class", ssl);
> + props.put("mail.smtp.socketFactory.fallback", "false");
> + }
> if (useSmtpAuth) {
> props.put("mail.smtp.auth", "true");
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
