Hi Vanji,
Thanks for the fix. Your PR was merged.
On Tue, Oct 4, 2016 at 6:15 AM, Vanjikumaran Sivajothy <va...@wso2.com>
wrote:
> Hi APIM team,
>
> $subject due to wrong use of method. It should be isNotEmpty[1]. I have
> fixed that and the send the pull request. Further i have introduced HTTP
> method in the log for more granular level of information to analyzing the
> log with log analyzing tools
>
>
> if (DIRECTION_OUT.equals(direction)) {
> String statusCode = String.valueOf(axisMC.
> getProperty(NhttpConstants.HTTP_SC));
>
> * if (StringUtils.isEmpty(statusCode)) {*
> logMessage = logMessage + " , statusCode=" + statusCode;
> }
> }
>
>
> [1] https://github.com/wso2/carbon-apimgt/blob/master/
> components/apimgt/org.wso2.carbon.apimgt.gateway/src/
> main/java/org/wso2/carbon/apimgt/gateway/handlers/
> logging/APILogMessageHandler.java#L126
>
>
> [2] https://github.com/wso2/carbon-apimgt/pull/2949
>
>
> --
> Vanjikumaran Sivajothy
> *Associate Technical Lead*
> *WSO2 Inc. http://wso2.com <http://wso2.com/>*
> *+1-925-464-6816*
> [image: Facebook] <https://www.facebook.com/vanjikumaran> [image: Twitter]
> <https://twitter.com/vanjikumaran> [image: LinkedIn]
> <http://www.linkedin.com/pub/vanjikumaran-sivajothy/25/b31/293> [image:
> Blogger] <http://vanjikumaran.blogspot.com/> [image: SlideShare]
> <http://www.slideshare.net/vanjikumaran>
>
> This communication may contain privileged or other
> confidential information and is intended exclusively for the addressee/s.
> If you are not the intended recipient/s, or believe that you may
> have received this communication in error, please reply to the
> sender indicating that fact and delete the copy you received and in
> addition, you should not print, copy, re-transmit, disseminate, or
> otherwise use the information contained in this communication.
> Internet communications cannot be guaranteed to be timely, secure, error
> or virus-free. The sender does not accept liability for any errors
> or omissions
>
> _______________________________________________
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>
--
Thanks
Abimaran Kugathasan
Senior Software Engineer - API Technologies
Email : abima...@wso2.com
Mobile : +94 773922820
<http://stackoverflow.com/users/515034>
<http://lk.linkedin.com/in/abimaran> <http://www.lkabimaran.blogspot.com/>
<https://github.com/abimarank> <https://twitter.com/abimaran>
_______________________________________________
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev
