ofbiz-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jacques Le Roux <jacques.le.r...@les7arts.com>
Subject Re: Sending mail from Ofbiz does not work
Date Fri, 16 Feb 2018 07:49:25 GMT
This could be a solution for this specific problem if we get a consensus.  OFBIZ-7754 is related

To summarize: the problem is, because of OFBIZ-7112, if you use the same seeds than in 13.07
you will get nothing which can even be more confusing.
That's why we have values in SystemProperty, this was done with r1748560.

While at it, and about OFBIZ-7754 what about the other SystemProperty in other seed or seed-initial
data files.
seed-initial: WorkEffortSeedInitialData CatalogSystemPropertyData OrderSystemPropertyData
BiSystemPropertyData ProjectMgrSystemPropertyData
seed: CommonSystemPropertyData EcommerceSystemPropertyData

I note that we have no other solutions yet than EntityUtilProperties to handle properties
in multi-tenants.
There is another related topic: we need to be sure to keep the SystemProperty and the properties
in file synchronised as shown in OFBIZ-9924
I wonder if a solution could not be to remove any property which has a related SystemProperty.
What do you think about that?

So we need to get a consensus, or even a vote if necessary, to definitely resolve these issues.

For that I exceptionally cross post this discussion in dev ML and it should be continued there.

Thanks

Jacques

Le 15/02/2018 à 18:22, Mike a écrit :
>>   but to comment them out of the ofbiz-component.xml.
> +1
>
> On Thu, Feb 15, 2018 at 8:42 AM, Michael Brohl <michael.brohl@ecomify.de>
> wrote:
>
>> I agree that the default population of SystemProperty with configuration
>> values is confusing, especially for the mail configuration
>>
>> I'd suggest to not remove the load data but to comment them out of the
>> ofbiz-component.xml. They can stay there as an example but would not be
>> loaded by default.
>>
>> Regards,
>>
>> Michael
>>
>>
>> Am 15.02.18 um 17:07 schrieb Mike:
>>
>>    Jacques: I understand the value of the feature.  What I'm referring to is
>>> somebody, in 16.x, hard-coded the above values in "seed", which caused the
>>> problem for this user.
>>>
>>> This is an advanced feature, and caused a lot of confusion.  I'd recommend
>>> that the 16.x CommonSystemPropertyData.xml be edited to remove all
>>> "systemPropertyValue="
>>> entries.
>>>
>>> 13.07: ./framework/common/data/CommonSystemPropertyData.xml
>>>
>>> Here is the latest version of 13.07, which does not hard-code these
>>> values.
>>> None of the 13.07 seed data have "systemPropertyValue=" set.
>>>
>>> systemPropertyId="ORGANIZATION_PARTY
>>> systemPropertyId="VISUAL_THEME"
>>> systemPropertyId="currency.uom.id.default"
>>> systemPropertyId="country.geo.id.default"
>>> systemPropertyId="partner.trackingCodeId.default"
>>> systemPropertyId="defaultFromEmailAddress"
>>> systemPropertyId="mail.notifications.enabled"
>>> systemPropertyId="mail.smtp.relay.host"
>>> systemPropertyId="mail.smtp.auth.user"
>>> systemPropertyId="mail.smtp.auth.password"
>>> systemPropertyId="mail.smtp.port"
>>> systemPropertyId="mail.smtp.starttls.enable"
>>> systemPropertyId="mail.smtp.socketFactory.port"
>>> systemPropertyId="mail.smtp.socketFactory.class"
>>> systemPropertyId="mail.smtp.socketFactory.fallback"
>>> systemPropertyId="mail.smtp.sendpartial"
>>>
>>>
>>> On Thu, Feb 15, 2018 at 1:15 AM, Jacques Le Roux <
>>> jacques.le.roux@les7arts.com> wrote:
>>>
>>> Mike, thanks for asking
>>>> This controversial feature has been initially discussed with
>>>> http://markmail.org/message/be3ts56b5w22k6pz
>>>>
>>>> We currently have some related pending Jira about that (sorry maybe a bit
>>>> too much, also a way to remind/check myself before discussing again in
>>>> dev
>>>> ML)
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-7112
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-7754
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-6166
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-6164
>>>>
>>>> http://markmail.org/message/i4rubhbo7wlm4wts
>>>>
>>>> https://s.apache.org/oTA6
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-6712
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-6205
>>>>
>>>> https://issues.apache.org/jira/browse/OFBIZ-6210
>>>>
>>>> Because this is now entrenched in OFBiz for many years, and I guess used
>>>> by many customs projects, it will maybe hard to get back.
>>>> But then we need a better documentation. Beginning as simply as I
>>>> proposed
>>>> below. And we need to agree and fix the pending issues.
>>>>
>>>> HTH
>>>>
>>>> Jacques
>>>>
>>>>
>>>>
>>>> Le 14/02/2018 à 16:49, Mike a écrit :
>>>>
>>>> Jacques:  Why does ofbiz 16.x set real properties
>>>>> in: ./framework/common/data/CommonSystemPropertyData.xml?  This is part
>>>>> of
>>>>> "seed"... It hard-codes:
>>>>>
>>>>>
>>>>> systemPropertyId="ORGANIZATION_PARTY" systemPropertyValue="Company"
>>>>> systemPropertyId="VISUAL_THEME" systemPropertyValue="FLAT_GREY"
>>>>> systemPropertyId="currency.uom.id.default" systemPropertyValue="USD"
>>>>> systemPropertyId="country.geo.id.default" systemPropertyValue="USA"
>>>>> systemPropertyId="defaultFromEmailAddress" systemPropertyValue="
>>>>> ofbiztest@example.com"
>>>>> systemPropertyId="mail.notifications.enabled" systemPropertyValue="N"
>>>>> systemPropertyId="mail.smtp.port" systemPropertyValue="465"
>>>>> systemPropertyId="mail.smtp.starttls.enable" systemPropertyValue="true"
>>>>> systemPropertyId="mail.smtp.socketFactory.port"
>>>>> systemPropertyValue="465"
>>>>> systemPropertyId="mail.smtp.socketFactory.class"
>>>>> systemPropertyValue="javax.net.ssl.SSLSocketFactory"
>>>>> systemPropertyId="mail.smtp.socketFactory.fallback"
>>>>> systemPropertyValue="false"
>>>>> systemPropertyId="mail.smtp.sendpartial" systemPropertyValue="true"
>>>>>
>>>>> Which seems to override general.properties.
>>>>>
>>>>>
>>>>> On Tue, Feb 13, 2018 at 6:55 AM, Jacques Le Roux <
>>>>> jacques.le.roux@les7arts.com> wrote:
>>>>>
>>>>> Thanks Pierre!
>>>>>
>>>>>> This is indeed something which is tricky for new users and even easily
>>>>>> forgettable in general.
>>>>>>
>>>>>> Before I post about SystemProperty and EntityUtilProperties on dev
ML,
>>>>>> I
>>>>>> want to suggest here that we put a comment at the top of each
>>>>>> properties
>>>>>> file as a reminder that the properties there could be overridden
in a
>>>>>> SystemProperty
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>> Le 12/02/2018 à 21:32, pierre.gaudin a écrit :
>>>>>>
>>>>>> Also, have a look at SystemProperty entity for key
>>>>>>
>>>>>>> mail.notifications.enabled
>>>>>>>
>>>>>>> Pierre
>>>>>>>
>>>>>>> On 12/02/2018 19:53, Mike wrote:
>>>>>>>
>>>>>>> For TLS (mail.smtp.starttls.enable=true ), use port 587
>>>>>>>
>>>>>>>> On Mon, Feb 12, 2018 at 4:37 AM, Дмитрий Цыганок
<dt@kapitanweb.ru>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>> Hello!
>>>>>>>>
>>>>>>>> I've deployed Ofbiz several times, but each time with the
right
>>>>>>>>> settings,
>>>>>>>>> email notifications are not sent.
>>>>>>>>>
>>>>>>>>> Here are my settings from /var/www/ofbiz/framework/commo
>>>>>>>>> n/config/general.
>>>>>>>>> properties:
>>>>>>>>>
>>>>>>>>> unique.instanceId=ofbiz1
>>>>>>>>> currency.uom.id.default=USD
>>>>>>>>> ORGANIZATION_PARTY=Company
>>>>>>>>> VISUAL_THEME=RAINBOWSTONE_SAPHIR
>>>>>>>>> currency.decimal.format=#,##0.00
>>>>>>>>> currency.rounding.default=10
>>>>>>>>> currency.scale.enabled=N
>>>>>>>>> locale.properties.fallback=en
>>>>>>>>> #locales.available=ar,de,en,es,fr,hi,it,nl,pt,ro,ru,th,zh
>>>>>>>>> #timeZones.available=US/Eastern,US/Central,US/
>>>>>>>>> Mountain,US/Pacific,US/Alaska,US/Hawaii
>>>>>>>>> country.geo.id.default=USA
>>>>>>>>> partner.trackingCodeId.default=
>>>>>>>>> usps.address.match=(^.*?p[\\. ]*o[\\. ]*box.*$)|(^.*?post.*?office.*
>>>>>>>>> ?box.*$)|((^|(^.*?
>>>>>>>>> ))r[\\. ]*r[\\. ]*(( +)|([0-9#]+)).*$)|(^.*?rural.*?route.*$)
>>>>>>>>> defaultFromEmailAddress=dmtsyganok@gmail.com
>>>>>>>>> mail.notifications.enabled=Y
>>>>>>>>> mail.notifications.redirectTo=dt@kapitanweb.ru
>>>>>>>>> mail.smtp.relay.host=smtp.gmail.com
>>>>>>>>> mail.smtp.auth.user=dmtsyganok@gmail.com
>>>>>>>>> mail.smtp.auth.password=*******
>>>>>>>>> mail.smtp.port=465
>>>>>>>>> mail.smtp.starttls.enable=true
>>>>>>>>> mail.smtp.socketFactory.port=465
>>>>>>>>> mail.smtp.socketFactory.class=javax.net.ssl.SSLSocketFactory
>>>>>>>>> mail.smtp.socketFactory.fallback=false
>>>>>>>>> mail.address.caseInsensitive=N
>>>>>>>>> mail.debug.on=N
>>>>>>>>> mail.smtp.sendpartial=true
>>>>>>>>> http.upload.max.sizethreshold=10240
>>>>>>>>> http.upload.tmprepository=runtime/tmp
>>>>>>>>> http.upload.max.size=-1
>>>>>>>>> mail.spam.name=X-Spam-Flag
>>>>>>>>> mail.spam.value=YES
>>>>>>>>> Ofbiz always issues this error in the logs and the mail
is not sent:
>>>>>>>>>
>>>>>>>>> " 2018-01-17 22:21:19,562 |OFBiz-JobQueue-1     |EmailServices
>>>>>>>>>          |I| Mail notifications disabled in general.properties;
mail
>>>>>>>>> with
>>>>>>>>> subject [test] not sent to addressee [ dt@kapitanweb.ru
  "
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> I also tried different mail accounts, but the result
is always the
>>>>>>>>> same.
>>>>>>>>>
>>>>>>>>> What could be the reason? Please help me to solve this
problem.
>>>>>>>>> Thank you very much in advance!
>>>>>>>>>
>>>>>>>>> ---
>>>>>>>>> Best Regards,
>>>>>>>>> Dmitriy
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>


Mime
View raw message