On Wed, Jul 26, 2017 at 10:39 PM, Harsha Thirimanna <hars...@wso2.com> wrote:
> > On Wed, Jul 26, 2017 at 9:31 PM, KasunG Gajasinghe <kas...@wso2.com> > wrote: > >> >> >> On Wed, Jul 26, 2017 at 9:07 PM, Johann Nallathamby <joh...@wso2.com> >> wrote: >> >>> >>> >>> On Wed, Jul 26, 2017 at 8:58 PM, Harsha Thirimanna <hars...@wso2.com> >>> wrote: >>> >>>> Hi All, >>>> >>>> I think we feel like both pros and cons because of these data file are >>>> located under the configurations. These are not actually config files and >>>> as Kasun said if these are under the config folder then anyone can feel to >>>> change and expect the changes in system after restarting. And same as Johan >>>> said, we can explain in the documentation clearly about the behaviour. >>>> >>>> As an alternative solution, can't we have such files in some different >>>> root folder but not under the config or deployment ,because of these files >>>> are not either deployable artifact or configs. Then users will not >>>> misunderstand about these files and there can be its own behaviour as what >>>> we have now. >>>> >>> >>>> *Harsha Thirimanna* >>>> *Associate Tech Lead | WSO2* >>>> >>>> Email: hars...@wso2.com >>>> Mob: +94715186770 <+94%2071%20518%206770> >>>> Blog: http://harshathirimanna.blogspot.com/ >>>> Twitter: http://twitter.com/harshathirimann >>>> Linked-In: linked-in: http://www.linkedin.com/pub/ha >>>> rsha-thirimanna/10/ab8/122 >>>> <http://wso2.com/signature> >>>> >>>> On Wed, Jul 26, 2017 at 5:21 PM, KasunG Gajasinghe <kas...@wso2.com> >>>> wrote: >>>> >>>>> Hi Johann, >>>>> >>>>> >>>>> On Wed, Jul 26, 2017 at 3:22 PM, Farasath Ahamed <farasa...@wso2.com> >>>>> wrote: >>>>> >>>>>> Hi, >>>>>> >>>>>> email-admin-config.xml has the default email templates that are used >>>>>> for Email notifications. >>>>>> Since we have a UI to add/update new Email templates I don't think >>>>>> the user has to go to the registry and do any modifications. >>>>>> >>>>>> >>>>>> Thanks, >>>>>> Farasath >>>>>> >>>>>> Farasath Ahamed >>>>>> Software Engineer, WSO2 Inc.; http://wso2.com >>>>>> Mobile: +94777603866 >>>>>> Blog: blog.farazath.com >>>>>> Twitter: @farazath619 <https://twitter.com/farazath619> >>>>>> <http://wso2.com/signature> >>>>>> >>>>>> >>>>>> >>>>>> On Wed, Jul 26, 2017 at 1:30 PM, Johann Nallathamby <joh...@wso2.com> >>>>>> wrote: >>>>>> >>>>>>> Hi Kasun, >>>>>>> >>>>>>> I don't think we need to do this because, >>>>>>> >>>>>>> 1. This is the model we follow for some other files as well. >>>>>>> claim-config.xml >>>>>>> identity-event.properties >>>>>>> >>>>>>> >>>>> Yes, I think Isura mentioned the same. Let me revise my original >>>>> query. The problem I was trying to address is: >>>>> >>>>> 1. There is a config file under repository/conf/ which is only read >>>>> during start-up. Since it is read once, should it really be in the >>>>> repository/conf/? >>>>> >>>>> 2. If a config file is there under repository/conf/, then users expect >>>>> any changes to it will be visible to users. It is not that intuitive for a >>>>> first time user IMO. >>>>> >>>> >>>>> As the product is quite matured, I think it may be better to look into >>>>> these aspects to make the first time user experience better. Hence, the >>>>> request for $subject. >>>>> >>>> >>> Agreed. We may be able to improve this. >>> >>> >> Cool.. :) My suggestion is to honor the changes to the file. So, if the >> file hash is changed, we re-upload the changes. >> >> MD5 hash can be stored in registry as a property along with the email >> templates. It is a simple change to [1] as I understand. >> > > But after some one change from the UI then it will update the changes in > registry. But it is not reflected in the file. Again if some one change in > the file, then registry will override the changes in registry that is > changed by the UI. Is this expected ? > > Yes, that happens. User need to stick to one. This is actually a problem we had in our platform for a while. Some of our configurations are duplicated in both file system and registry. This leads to much confusion for a first time user. What's your suggestion? Thanks. > >> [1] https://github.com/wso2-extensions/identity-event-handler- >> notification/blob/master/components/email-mgt/org.wso2. >> carbon.email.mgt/src/main/java/org/wso2/carbon/email/ >> mgt/EmailTemplateManagerImpl.java#L78 >> >> >>> >>>>> >>>>> >>>>>> 2. This content goes not only to the super tenant but also to tenants >>>>>>> as default email templates. So if we change it in the middle tenants >>>>>>> created until then will have a different default template and the ones >>>>>>> created after that will have a different default template. >>>>>>> >>>>>>> >>>>> Yes, this is the current behavior, isn't it? >>>>> >>>> >>> Yes. This is the current behavior. >>> >>> >>>> >>>>> Thanks for your feedback. >>>>> >>>>> >>>>>> I don't think we have any problems with current model. If we do this >>>>>>> we need to change our migration, etc. If the issue here is clarity for >>>>>>> the >>>>>>> users it is better we explain it using better wordings in the docs. >>>>>>> >>>>>>> Regards, >>>>>>> Johann. >>>>>>> >>>>>>> ---------- Forwarded message ---------- >>>>>>> From: Kasun Gajasinghe (JIRA) <j...@wso2.org> >>>>>>> Date: Wed, Jul 19, 2017 at 1:54 AM >>>>>>> Subject: [jira] (IDENTITY-6167) Need to honor modifications to >>>>>>> email-admin-config.xml file >>>>>>> To: joh...@wso2.com >>>>>>> >>>>>>> >>>>>>> Kasun Gajasinghe >>>>>>> <https://wso2.org/jira/secure/ViewProfile.jspa?name=kasung%40wso2.com> >>>>>>> *created* an issue >>>>>>> >>>>>>> WSO2 Identity Server <https://wso2.org/jira/browse/IDENTITY> / [image: >>>>>>> Improvement] <https://wso2.org/jira/browse/IDENTITY-6167> >>>>>>> IDENTITY-6167 <https://wso2.org/jira/browse/IDENTITY-6167> >>>>>>> Need to honor modifications to email-admin-config.xml file >>>>>>> <https://wso2.org/jira/browse/IDENTITY-6167> >>>>>>> Issue Type: [image: Improvement] Improvement >>>>>>> Affects Versions: 5.3.0-GA >>>>>>> Assignee: Johann Nallathamby >>>>>>> <https://wso2.org/jira/secure/ViewProfile.jspa?name=johann%40wso2.com> >>>>>>> Created: 19/Jul/17 1:53 AM >>>>>>> Fix Versions: 5.4.0-GA >>>>>>> Priority: [image: High] High >>>>>>> Reporter: Kasun Gajasinghe >>>>>>> <https://wso2.org/jira/secure/ViewProfile.jspa?name=kasung%40wso2.com> >>>>>>> >>>>>>> At the moment, we save the email-admin-config.xml in the registry >>>>>>> during the very first server startup. After that, changes to the file >>>>>>> are >>>>>>> not picked up. So, users are forced to go to the registry to update the >>>>>>> config. >>>>>>> >>>>>>> This is a bit confusing since a user probably expects that the >>>>>>> changes to the file will be honored especially since it is located under >>>>>>> repository/conf folder. >>>>>>> >>>>>>> We can do some hashing mechanism and verify whether the contents in >>>>>>> the registry and the file matches. And, then we can update the registry >>>>>>> accordingly. >>>>>>> [image: Add Comment] >>>>>>> <https://wso2.org/jira/browse/IDENTITY-6167#add-comment> Add Comment >>>>>>> <https://wso2.org/jira/browse/IDENTITY-6167#add-comment> >>>>>>> >>>>>>> This message was sent by Atlassian JIRA (v7.2.2#72004-sha1:9d51328) >>>>>>> [image: Atlassian logo] >>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Thanks & Regards, >>>>>>> >>>>>>> *Johann Dilantha Nallathamby* >>>>>>> Senior Lead Solutions Engineer >>>>>>> WSO2, Inc. >>>>>>> lean.enterprise.middleware >>>>>>> >>>>>>> Mobile - *+94777776950* >>>>>>> Blog - *http://nallaa.wordpress.com <http://nallaa.wordpress.com>* >>>>>>> >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> >>>>> *Kasun Gajasinghe*Associate Technical Lead, WSO2 Inc. >>>>> email: kasung AT spamfree wso2.com >>>>> linked-in: http://lk.linkedin.com/in/gajasinghe >>>>> blog: http://kasunbg.org >>>>> phone: +1 650-745-4499 <(650)%20745-4499>, 77 678 0813 >>>>> >>>>> >>>>> _______________________________________________ >>>>> Dev mailing list >>>>> Dev@wso2.org >>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev >>>>> >>>>> >>>> >>> >>> >>> -- >>> Thanks & Regards, >>> >>> *Johann Dilantha Nallathamby* >>> Senior Lead Solutions Engineer >>> WSO2, Inc. >>> lean.enterprise.middleware >>> >>> Mobile - *+94777776950* >>> Blog - *http://nallaa.wordpress.com <http://nallaa.wordpress.com>* >>> >> >> >> >> -- >> >> *Kasun Gajasinghe*Associate Technical Lead, WSO2 Inc. >> email: kasung AT spamfree wso2.com >> linked-in: http://lk.linkedin.com/in/gajasinghe >> blog: http://kasunbg.org >> phone: +1 650-745-4499 <(650)%20745-4499>, 77 678 0813 >> >> > > -- *Kasun Gajasinghe*Associate Technical Lead, WSO2 Inc. email: kasung AT spamfree wso2.com linked-in: http://lk.linkedin.com/in/gajasinghe blog: http://kasunbg.org phone: +1 650-745-4499, 77 678 0813
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev