Hi All,
The original issue encountered in [1] was fixed via the resolution of [2].
But now this has been reopened and in this case the same issue has got
exposed, but on this occasion the call stack is different. This is
happening when setting the tenant info at [3]. So we need to apply the same
Hi Uvindra,
Could you please verify the fix and send us a PR so that we can merge it?
On Fri, Jun 24, 2016 at 1:02 PM, Uvindra Dias Jayasinha
wrote:
> Hi All,
>
> The original issue encountered in [1] was fixed via the resolution of [2].
>
> But now this has been reopened and in this case the s
Problem is the issue is intermittent so its difficult to verify but I will
test the fix out and send a PR
On 24 June 2016 at 13:52, Chanaka Fernando wrote:
> Hi Uvindra,
>
> Could you please verify the fix and send us a PR so that we can merge it?
>
> On Fri, Jun 24, 2016 at 1:02 PM, Uvindra Dia
Before I just apply the stated fix I would like some feedback from the
Carbon team regarding what maybe causing this issue.
So here is what we are seeing so far,
1. IllegalStateException is being thrown by
CarbonContextDataHolder.setTenantDomain()
2. The logic in setTenantDomain() is as follows,
Ping! Need your feedback Carbon team.
On 27 June 2016 at 12:13, Uvindra Dias Jayasinha wrote:
> Before I just apply the stated fix I would like some feedback from the
> Carbon team regarding what maybe causing this issue.
>
> So here is what we are seeing so far,
>
> 1. IllegalStateException is
On Mon, Jun 27, 2016 at 12:13 PM, Uvindra Dias Jayasinha
wrote:
> Before I just apply the stated fix I would like some feedback from the
> Carbon team regarding what maybe causing this issue.
>
> So here is what we are seeing so far,
>
> 1. IllegalStateException is being thrown by
> CarbonContext
Thanks Kishanthan
If we were to fix this by calling startTenantFlow() the fix would be a lot
more complicated and may lead to other issues. So I have tested the
original fix done by Jagath in this case and didn't run into any issues
functional wise. Though I could not recreate the issue in the fir
Ping!
ESB Team can we get the above PR merged please?
On 27 June 2016 at 19:26, Uvindra Dias Jayasinha wrote:
> Thanks Kishanthan
>
> If we were to fix this by calling startTenantFlow() the fix would be a lot
> more complicated and may lead to other issues. So I have tested the
> original fix d
PR is merged https://github.com/wso2/carbon-mediation/pull/671.
On Tue, Jul 5, 2016 at 3:44 PM, Uvindra Dias Jayasinha
wrote:
> Ping!
>
> ESB Team can we get the above PR merged please?
>
> On 27 June 2016 at 19:26, Uvindra Dias Jayasinha wrote:
>
>> Thanks Kishanthan
>>
>> If we were to fix th
Thanks Viraj
On 5 July 2016 at 15:52, Viraj Senevirathne wrote:
> PR is merged https://github.com/wso2/carbon-mediation/pull/671.
>
> On Tue, Jul 5, 2016 at 3:44 PM, Uvindra Dias Jayasinha
> wrote:
>
>> Ping!
>>
>> ESB Team can we get the above PR merged please?
>>
>> On 27 June 2016 at 19:26,
10 matches
Mail list logo