[jira] [Reopened] (OFBIZ-10879) updateCommEventAfterEmail: Restore original sendTo/sendCc/sendBcc recipients from mail subject for general redirected mails

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Reopened] (OFBIZ-10879) updateCommEventAfterEmail: Restore original sendTo/sendCc/sendBcc recipients from mail subject for general redirected mails

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-10879?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Deepak Dixit reopened OFBIZ-10879:
----------------------------------

I think this is not the right fix for reported problems.As we are updating so we don't need to pass the to,cc and bcc. And redirectTo will be used in dev mode only.We can fix the subject length only. 
IMO we should revert this. 

> updateCommEventAfterEmail: Restore original sendTo/sendCc/sendBcc recipients from mail subject for general redirected mails
> ---------------------------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-10879
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10879
>             Project: OFBiz
>          Issue Type: Bug
>    Affects Versions: Trunk
>            Reporter: Ulrich Heidfeld
>            Assignee: Pawan Verma
>            Priority: Minor
>             Fix For: 17.12.01, Upcoming Branch, 18.12.01
>
>         Attachments: OFBIZ-10879.patch, OFBIZ-10879.patch
>
>
> "updateCommEventAfterEmail" method in CommunicationEventServices.java could generate  wrong result. If general.properties value "mail.notifications.redirectTo" is used, it is possible that the subject string has more than 255 character.
> Now:
> {code:java}
> Original subject [To: <sendTo>, Cc: <sendCc>, Bcc: <sendBcc>]
> {code}
> Solution:
> {code:java}
> Original subject [RedirectedTo: [hidden email]]
> {code}
> In this way we have fixed this in one of our OFBiz forks.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)