[GitHub] [synapse] isudana merged pull request #50: Improving how Target Endpoint is Associated with Message Forwarding Processor

isudana merged pull request #50: URL: https://github.com/apache/synapse/pull/50 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: Improving how Target Endpoint is Associated with Message Forwarding Processor

Hi Isuru, I have updated PR as per the comments. On Mon, Nov 2, 2020 at 9:07 PM Shafreen wrote: > Thanks. Will update the PR accordingly. > > On Mon, Nov 2, 2020 at 7:53 AM Isuru Udana wrote: > >> Thanks Shafreen. I reviewed it and suggested a few minor modifications. >> >> On Sun, Nov 1, 2020

Re: Improving how Target Endpoint is Associated with Message Forwarding Processor

Thanks. Will update the PR accordingly. On Mon, Nov 2, 2020 at 7:53 AM Isuru Udana wrote: > Thanks Shafreen. I reviewed it and suggested a few minor modifications. > > On Sun, Nov 1, 2020 at 11:42 AM Shafreen wrote: > >> Hi All, >> >> Please find the below PR for this improvement. >> >> https:/

Re: Improving how Target Endpoint is Associated with Message Forwarding Processor

Thanks Shafreen. I reviewed it and suggested a few minor modifications. On Sun, Nov 1, 2020 at 11:42 AM Shafreen wrote: > Hi All, > > Please find the below PR for this improvement. > > https://github.com/apache/synapse/pull/50 > > I will send another PR updating the doc once this is merged. > >

Re: Improving how Target Endpoint is Associated with Message Forwarding Processor

Hi All, Please find the below PR for this improvement. https://github.com/apache/synapse/pull/50 I will send another PR updating the doc once this is merged. On Sat, Oct 17, 2020 at 9:27 AM Vanjikumaran Sivajothy < vanjikuma...@gmail.com> wrote: > I agree with your suggestion; it is much bette

Re: Improving how Target Endpoint is Associated with Message Forwarding Processor

I agree with your suggestion; it is much better clean design approach. On Fri, Oct 16, 2020 at 8:34 PM Shafreen wrote: > Hi All, > > Any opinion on this? > > On Sun, Oct 11, 2020 at 11:45 AM Shafreen > wrote: > >> Hi All, >> >> Each Message Forwarding Processor must have a target endpoint. At

Re: Improving how Target Endpoint is Associated with Message Forwarding Processor

Hi All, Any opinion on this? On Sun, Oct 11, 2020 at 11:45 AM Shafreen wrote: > Hi All, > > Each Message Forwarding Processor must have a target endpoint. At the > moment we configure this target endpoint by adding a property in the > in-sequence of proxy service as follows. > > > > IMO, this

Improving how Target Endpoint is Associated with Message Forwarding Processor

Hi All, Each Message Forwarding Processor must have a target endpoint. At the moment we configure this target endpoint by adding a property in the in-sequence of proxy service as follows. IMO, this way of associating target-endpoint with Message Forwarding Processor is not only error prone but