ofbiz-user 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 Sun, 18 Feb 2018 07:11:08 GMT
As I recommended I have answered in the dev ML: https://s.apache.org/Hseo

Jacques


Le 17/02/2018 à 19:47, Mike a écrit :
> Once more rule:
>
>
> If the SystemProperty table has a systemPropertyID called "ConfigFile"
> and "systemPropertyValue"
> path exists, and the timestamp of the file matches last_updated_stamp in
> the DB, the DB has precedence.  No sync needed.
>
> On Sat, Feb 17, 2018 at 9:52 AM, Mike <mz4wheeler@gmail.com> wrote:
>
>> The way it is implemented now violates the "do no harm" principle, which
>> includes years of documentation (and books) that supports the traditional
>> use of java property files.  The user has said that he has tried off/on for
>> 6 months to get email notifications to work. In fact, I didn't even know
>> about this 16.x behavior.   Here is a suggestion:
>>
>> Assumes seeded via CommonSystemPropertyData.xml... Or other, similar
>> XXX_propertyData.xml files...
>>
>> If the SystemProperty table has a systemPropertyID called "ConfigFile" and
>> "systemPropertyValue" path exists, then the property file has precedence,
>> and always does a 1-way sync to the DB.
>>
>> If the SystemProperty table does not have "ConfigFile" key, then do
>> nothing.  The DB has precedence.
>>
>> If the SystemProperty table has a "ConfigFile" entry, and "systemPropertyValue"
>> path does NOT exist, the system will create it once, and from that point
>> forward the property file is referenced for future loads.
>>
>> Example:
>>
>> <entity-engine-xml>
>>      <SystemProperty systemResourceId="general"
>> systemPropertyId="ConfigFile" systemPropertyValue="
>> framework/common/config/general.properties" description="config file used
>> for general"/>
>>
>> By default (seed), the property file should exist and have precedence.
>> If you are doing advanced stuff like multi-tenant, then you need to be more
>> vigilant and set "ConfigFile" and it's path more carefully in the
>> SystemProperty table.
>>
>>
>> On Thu, Feb 15, 2018 at 11:49 PM, Jacques Le Roux <
>> jacques.le.roux@les7arts.com> wrote:
>>
>>> 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