Hi All,
We have discussed with the Kernel team and they have also suggested not use
${carbon.management.port}. We'll change to support ${mgt.transport.https.port}
and ${mgt.transport.http.port} placeholders instead of ${
carbon.management.port}.
Thanks!
*Jayanga Kaushalya*
Software Engineer
Mobile: +94777860160
WSO2 Inc. | http://wso2.com
lean.enterprise.middleware
On Thu, Dec 3, 2015 at 12:10 PM, Nuwan Dias <nuw...@wso2.com> wrote:
> Since there is a parameter available in the kernel, can't we use the same
> thing rather than using a new one?
>
> BTW, this will be an issue for ESB and all other products having custom
> transports defined in the axis2.xml file when they try to use identity
> related features.
>
> Thanks,
> NuwanD.
>
> On Thu, Dec 3, 2015 at 11:23 AM, Jayanga Kaushalya <jayan...@wso2.com>
> wrote:
>
>> Hi Lakmali,
>>
>> This port is used in the Kernal and we have to check why it is returning
>> the NIO port. We are using the same code as in Kernal to get the value for
>> this. Will check with the Kernal team and inform further.
>>
>> Thanks!
>>
>> *Jayanga Kaushalya*
>> Software Engineer
>> Mobile: +94777860160
>> WSO2 Inc. | http://wso2.com
>> lean.enterprise.middleware
>>
>> On Thu, Dec 3, 2015 at 10:55 AM, Lakmali Baminiwatta <lakm...@wso2.com>
>> wrote:
>>
>>> Hi IS Team,
>>>
>>> The variable "${carbon.management.port}" used in identity.xml is getting
>>> evaluated to NIO port in APIM. We can see that IdentitUtil class[1]
>>> resolves this variable.
>>> I think instead of using your own variable here, we can use
>>> "mgt.transport.https.port" system variable which is already available. So
>>> you can just use System.getProperty("mgt.transport.https.port") and
>>> System.getProperty("mgt.transport.http.port") [2] , which will return the
>>> servlet HTTPS and HTTP ports.
>>>
>>> [1]
>>> https://github.com/wso2/carbon-identity/blame/master/components/identity-core/org.wso2.carbon.identity.core/src/main/java/org/wso2/carbon/identity/core/util/IdentityUtil.java
>>> [2]
>>> https://github.com/wso2/carbon-kernel/blob/4.4.x/core/org.wso2.carbon.tomcat.ext/src/main/java/org/wso2/carbon/tomcat/ext/transport/ServletTransportManager.java#L34
>>>
>>> Thanks,
>>> Lakmali
>>>
>>> --
>>> Lakmali Baminiwatta
>>> Senior Software Engineer
>>> WSO2, Inc.: http://wso2.com
>>> lean.enterprise.middleware
>>> mobile: +94 71 2335936
>>> blog : lakmali.com
>>>
>>>
>>> _______________________________________________
>>> Dev mailing list
>>> Dev@wso2.org
>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>
>>>
>>
>> _______________________________________________
>> Dev mailing list
>> Dev@wso2.org
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>
>>
>
>
> --
> Nuwan Dias
>
> Technical Lead - WSO2, Inc. http://wso2.com
> email : nuw...@wso2.com
> Phone : +94 777 775 729
>
_______________________________________________
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev
