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. [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, 77 678 0813
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev