0013957: To Configure SMTP Sever Port - Openbravo Issue Tracking System

diff --git a/src-db/database/sourcedata/AD_COLUMN.xml b/src-db/database/sourcedata/AD_COLUMN.xml --- a/src-db/database/sourcedata/AD_COLUMN.xml +++ b/src-db/database/sourcedata/AD_COLUMN.xml @@ -265651,6 +265651,37 @@  <VALIDATEONNEW><![CDATA[Y]]></VALIDATEONNEW> </AD_COLUMN> +<AD_COLUMN> + <AD_COLUMN_ID><![CDATA[2AFA0BE438424C6881A0A52781FF304E]]></AD_COLUMN_ID> + <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> + <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID> + <ISACTIVE><![CDATA[Y]]></ISACTIVE> + <NAME><![CDATA[Use SSL]]></NAME> + <COLUMNNAME><![CDATA[Usessl]]></COLUMNNAME> + <AD_TABLE_ID><![CDATA[1000500000]]></AD_TABLE_ID> + <AD_REFERENCE_ID><![CDATA[20]]></AD_REFERENCE_ID> + <FIELDLENGTH><![CDATA[1]]></FIELDLENGTH> + <DEFAULTVALUE><![CDATA[N]]></DEFAULTVALUE> + <ISKEY><![CDATA[N]]></ISKEY> + <ISPARENT><![CDATA[N]]></ISPARENT> + <ISMANDATORY><![CDATA[N]]></ISMANDATORY> + <ISUPDATEABLE><![CDATA[Y]]></ISUPDATEABLE> + <ISIDENTIFIER><![CDATA[N]]></ISIDENTIFIER> + <SEQNO><![CDATA[170]]></SEQNO> + <ISTRANSLATED><![CDATA[N]]></ISTRANSLATED> + <ISENCRYPTED><![CDATA[N]]></ISENCRYPTED> + <ISSELECTIONCOLUMN><![CDATA[N]]></ISSELECTIONCOLUMN> + <AD_ELEMENT_ID><![CDATA[4796B4D47ACE4CF480DA713D1F261A9A]]></AD_ELEMENT_ID> + <ISSESSIONATTR><![CDATA[N]]></ISSESSIONATTR> + <ISSECONDARYKEY><![CDATA[N]]></ISSECONDARYKEY> + <ISDESENCRYPTABLE><![CDATA[N]]></ISDESENCRYPTABLE> + <DEVELOPMENTSTATUS><![CDATA[RE]]></DEVELOPMENTSTATUS> + <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> + <ISTRANSIENT><![CDATA[N]]></ISTRANSIENT> + <ISAUTOSAVE><![CDATA[Y]]></ISAUTOSAVE> + <VALIDATEONNEW><![CDATA[Y]]></VALIDATEONNEW> +</AD_COLUMN> + <AD_COLUMN>  <AD_COLUMN_ID><![CDATA[2B5F702BA50F4C2ABB8E14B77F8BE893]]></AD_COLUMN_ID>  <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> @@ -294836,6 +294867,36 @@  <VALIDATEONNEW><![CDATA[Y]]></VALIDATEONNEW> </AD_COLUMN> +<AD_COLUMN> + <AD_COLUMN_ID><![CDATA[7F93962065E44390A89334F7E82A6B49]]></AD_COLUMN_ID> + <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> + <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID> + <ISACTIVE><![CDATA[Y]]></ISACTIVE> + <NAME><![CDATA[SMTP Server Port]]></NAME> + <COLUMNNAME><![CDATA[Smtpserverport]]></COLUMNNAME> + <AD_TABLE_ID><![CDATA[1000500000]]></AD_TABLE_ID> + <AD_REFERENCE_ID><![CDATA[11]]></AD_REFERENCE_ID> + <FIELDLENGTH><![CDATA[4]]></FIELDLENGTH> + <ISKEY><![CDATA[N]]></ISKEY> + <ISPARENT><![CDATA[N]]></ISPARENT> + <ISMANDATORY><![CDATA[N]]></ISMANDATORY> + <ISUPDATEABLE><![CDATA[Y]]></ISUPDATEABLE> + <ISIDENTIFIER><![CDATA[N]]></ISIDENTIFIER> + <SEQNO><![CDATA[160]]></SEQNO> + <ISTRANSLATED><![CDATA[N]]></ISTRANSLATED> + <ISENCRYPTED><![CDATA[N]]></ISENCRYPTED> + <ISSELECTIONCOLUMN><![CDATA[N]]></ISSELECTIONCOLUMN> + <AD_ELEMENT_ID><![CDATA[C17E4AB475B0408A881DCE757CC0B05B]]></AD_ELEMENT_ID> + <ISSESSIONATTR><![CDATA[N]]></ISSESSIONATTR> + <ISSECONDARYKEY><![CDATA[N]]></ISSECONDARYKEY> + <ISDESENCRYPTABLE><![CDATA[N]]></ISDESENCRYPTABLE> + <DEVELOPMENTSTATUS><![CDATA[RE]]></DEVELOPMENTSTATUS> + <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> + <ISTRANSIENT><![CDATA[N]]></ISTRANSIENT> + <ISAUTOSAVE><![CDATA[Y]]></ISAUTOSAVE> + <VALIDATEONNEW><![CDATA[Y]]></VALIDATEONNEW> +</AD_COLUMN> + <AD_COLUMN>  <AD_COLUMN_ID><![CDATA[7FCA79F6A6804336A083FEE0C8964D6E]]></AD_COLUMN_ID>  <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> diff --git a/src-db/database/model/tables/C_POC_CONFIGURATION.xml b/src-db/database/model/tables/C_POC_CONFIGURATION.xml --- a/src-db/database/model/tables/C_POC_CONFIGURATION.xml +++ b/src-db/database/model/tables/C_POC_CONFIGURATION.xml @@ -53,6 +53,14 @@ <default/> <onCreateDefault/> </column> + <column name="SMTPSERVERPORT" primaryKey="false" required="false" type="DECIMAL" size="8,0" autoIncrement="false"> + <default/> + <onCreateDefault/> + </column> + <column name="USESSL" primaryKey="false" required="false" type="CHAR" size="1" autoIncrement="false"> + <default/> + <onCreateDefault/> + </column> <foreign-key foreignTable="AD_CLIENT" name="C_POC_CONFIGURATION_AD_CLIENT"> <reference local="AD_CLIENT_ID" foreign="AD_CLIENT_ID"/> </foreign-key> diff --git a/src-db/database/sourcedata/AD_FIELD.xml b/src-db/database/sourcedata/AD_FIELD.xml --- a/src-db/database/sourcedata/AD_FIELD.xml +++ b/src-db/database/sourcedata/AD_FIELD.xml @@ -207551,6 +207551,29 @@  <STARTNEWLINE><![CDATA[N]]></STARTNEWLINE> </AD_FIELD> +<AD_FIELD> + <AD_FIELD_ID><![CDATA[5A50004544124E469BE1FB083B32BAE1]]></AD_FIELD_ID> + <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> + <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID> + <ISACTIVE><![CDATA[Y]]></ISACTIVE> + <NAME><![CDATA[SMTP Server Port]]></NAME> + <ISCENTRALLYMAINTAINED><![CDATA[Y]]></ISCENTRALLYMAINTAINED> + <AD_TAB_ID><![CDATA[1000500000]]></AD_TAB_ID> + <AD_COLUMN_ID><![CDATA[7F93962065E44390A89334F7E82A6B49]]></AD_COLUMN_ID> + <ISDISPLAYED><![CDATA[Y]]></ISDISPLAYED> + <DISPLAYLENGTH><![CDATA[0]]></DISPLAYLENGTH> + <ISREADONLY><![CDATA[N]]></ISREADONLY> + <SEQNO><![CDATA[90]]></SEQNO> + <ISSAMELINE><![CDATA[N]]></ISSAMELINE> + <ISFIELDONLY><![CDATA[N]]></ISFIELDONLY> + <ISENCRYPTED><![CDATA[N]]></ISENCRYPTED> + <SHOWINRELATION><![CDATA[N]]></SHOWINRELATION> + <ISFIRSTFOCUSEDFIELD><![CDATA[N]]></ISFIRSTFOCUSEDFIELD> + <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> + <STARTINODDCOLUMN><![CDATA[N]]></STARTINODDCOLUMN> + <STARTNEWLINE><![CDATA[N]]></STARTNEWLINE> +</AD_FIELD> + <AD_FIELD>  <AD_FIELD_ID><![CDATA[5A514F7613262417E040007F010049A1]]></AD_FIELD_ID>  <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> @@ -224035,6 +224058,29 @@  <STARTNEWLINE><![CDATA[N]]></STARTNEWLINE> </AD_FIELD> +<AD_FIELD> + <AD_FIELD_ID><![CDATA[FCCD257936504D068A023F6AE84C61EA]]></AD_FIELD_ID> + <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> + <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID> + <ISACTIVE><![CDATA[Y]]></ISACTIVE> + <NAME><![CDATA[Use SSL]]></NAME> + <ISCENTRALLYMAINTAINED><![CDATA[Y]]></ISCENTRALLYMAINTAINED> + <AD_TAB_ID><![CDATA[1000500000]]></AD_TAB_ID> + <AD_COLUMN_ID><![CDATA[2AFA0BE438424C6881A0A52781FF304E]]></AD_COLUMN_ID> + <ISDISPLAYED><![CDATA[Y]]></ISDISPLAYED> + <DISPLAYLENGTH><![CDATA[0]]></DISPLAYLENGTH> + <ISREADONLY><![CDATA[N]]></ISREADONLY> + <SEQNO><![CDATA[100]]></SEQNO> + <ISSAMELINE><![CDATA[N]]></ISSAMELINE> + <ISFIELDONLY><![CDATA[N]]></ISFIELDONLY> + <ISENCRYPTED><![CDATA[N]]></ISENCRYPTED> + <SHOWINRELATION><![CDATA[N]]></SHOWINRELATION> + <ISFIRSTFOCUSEDFIELD><![CDATA[N]]></ISFIRSTFOCUSEDFIELD> + <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> + <STARTINODDCOLUMN><![CDATA[N]]></STARTINODDCOLUMN> + <STARTNEWLINE><![CDATA[N]]></STARTNEWLINE> +</AD_FIELD> + <AD_FIELD>  <AD_FIELD_ID><![CDATA[FF6017DC061144718EB362D7DE06A614]]></AD_FIELD_ID>  <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> diff --git a/src-db/database/sourcedata/AD_ELEMENT.xml b/src-db/database/sourcedata/AD_ELEMENT.xml --- a/src-db/database/sourcedata/AD_ELEMENT.xml +++ b/src-db/database/sourcedata/AD_ELEMENT.xml @@ -22313,6 +22313,17 @@  <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> </AD_ELEMENT> +<AD_ELEMENT> + <AD_ELEMENT_ID><![CDATA[4796B4D47ACE4CF480DA713D1F261A9A]]></AD_ELEMENT_ID> + <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> + <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID> + <ISACTIVE><![CDATA[Y]]></ISACTIVE> + <COLUMNNAME><![CDATA[Usessl]]></COLUMNNAME> + <NAME><![CDATA[Use SSL]]></NAME> + <PRINTNAME><![CDATA[Use SSL]]></PRINTNAME> + <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> +</AD_ELEMENT> + <AD_ELEMENT>  <AD_ELEMENT_ID><![CDATA[4823F0304FFC4CBF84A17244F0431117]]></AD_ELEMENT_ID>  <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> @@ -27777,6 +27788,17 @@  <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> </AD_ELEMENT> +<AD_ELEMENT> + <AD_ELEMENT_ID><![CDATA[C17E4AB475B0408A881DCE757CC0B05B]]></AD_ELEMENT_ID> + <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> + <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID> + <ISACTIVE><![CDATA[Y]]></ISACTIVE> + <COLUMNNAME><![CDATA[Smtpserverport]]></COLUMNNAME> + <NAME><![CDATA[SMTP Server Port]]></NAME> + <PRINTNAME><![CDATA[SMTP Server Port]]></PRINTNAME> + <AD_MODULE_ID><![CDATA[0]]></AD_MODULE_ID> +</AD_ELEMENT> + <AD_ELEMENT>  <AD_ELEMENT_ID><![CDATA[CB65CBB36988405DA4D2F2C36CD29DC4]]></AD_ELEMENT_ID>  <AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID> diff --git a/src/org/openbravo/erpCommon/utility/poc/EmailManager.java b/src/org/openbravo/erpCommon/utility/poc/EmailManager.java --- a/src/org/openbravo/erpCommon/utility/poc/EmailManager.java +++ b/src/org/openbravo/erpCommon/utility/poc/EmailManager.java @@ -62,16 +62,25 @@ log4j.debug("Crm configuration, smtp server account: " + configuration.smtpserveraccount); if (log4j.isDebugEnabled()) log4j.debug("Crm configuration, smtp server password: " + configuration.smtpserverpassword); + log4j.debug("Crm configuration, smtp server port: " + configuration.smtpserverport); + log4j.debug("Crm configuration, smtp server ssl: '" + configuration.usessl +"'"); } else { throw new ServletException("No Poc configuration found for this client."); } - + Properties props = new Properties(); props.put("mail.debug", "true"); props.put("mail.smtp.auth", (configuration.issmtpauthorization.equals("Y") ? "true" : "false")); props.put("mail.transport.protocol", "smtp"); + props.put("mail.host", configuration.smtpserver); props.put("mail.smtp.mail.sender", "email_admin@openbravo.com"); - props.put("mail.host", configuration.smtpserver); + if (configuration.smtpserverport != null && configuration.smtpserverport != "") { + props.put("mail.smtp.socketFactory.port", configuration.smtpserverport); + } + if (configuration.usessl != null && configuration.usessl.equals("Y")) { + props.put("mail.smtp.socketFactory.class", "javax.net.ssl.SSLSocketFactory"); + props.put("mail.smtp.starttls.enable","true"); + } ClientAuthenticator authenticator = null; if (configuration.smtpserveraccount != null) { diff --git a/src/org/openbravo/erpCommon/utility/poc/Poc_Configuration_data.xsql b/src/org/openbravo/erpCommon/utility/poc/Poc_Configuration_data.xsql --- a/src/org/openbravo/erpCommon/utility/poc/Poc_Configuration_data.xsql +++ b/src/org/openbravo/erpCommon/utility/poc/Poc_Configuration_data.xsql @@ -35,7 +35,9 @@ '' as smtpServerAccount, '' as smtpServerPassword, '' as isSmtpAuthorization, - '' as smtpServerSenderAddress + '' as smtpServerSenderAddress, + '' as smtpServerPort, + '' as useSsl from dual ]]> @@ -51,7 +53,9 @@ C_poc_configuration.smtpServerAccount, C_poc_configuration.smtpServerPassword, C_poc_configuration.isSmtpAuthorization, - C_poc_configuration.smtpServerSenderAddress + C_poc_configuration.smtpServerSenderAddress, + C_poc_configuration.smtpServerPort, + C_poc_configuration.useSsl from C_poc_configuration where

Issue History
Date Modified	Username	Field	Change
2010-07-14 15:59	sengkin	New Issue
2010-07-14 15:59	sengkin	Assigned To	=> adrianromero
2011-01-26 16:46	roklenardic	File Added: email_ssl_config.diff
2011-01-26 16:47	roklenardic	Note Added: 0033882
2011-01-27 10:18	rafaroda	Issue Monitored: rafaroda
2011-01-27 11:52	shuehner	Assigned To	adrianromero => shuehner
2011-10-06 11:18	roklenardic	Note Added: 0041526
2011-10-06 11:22	RenateNieuwkoop	Issue Monitored: RenateNieuwkoop
2012-01-05 16:32	shuehner	Relationship added	related to 0019406
2012-01-05 16:43	shuehner	Assigned To	shuehner => dbaz
2012-01-10 13:15	dbaz	Relationship replaced	duplicate of 0019406
2012-01-10 13:15	dbaz	Note Added: 0044175
2012-01-10 13:15	dbaz	Status	new => closed
2012-01-10 13:15	dbaz	Resolution	open => duplicate

Notes
(0033882) roklenardic (developer) 2011-01-26 16:47	Submitted the patch for review and inclusion in the RC4 core based on PI from January 25th, 2011.

(0041526) roklenardic (developer) 2011-10-06 11:18	Why has nothing happened to this issue? Does the patch need modifications?

(0044175) dbaz (developer) 2012-01-10 13:15	The issue 19406 has a proper implementation.