Fix Sonar Issues 59/11859/2
authorsurya-huawei <a.u.surya@huawei.com>
Tue, 12 Sep 2017 06:22:59 +0000 (11:52 +0530)
committerSURYA A U <a.u.surya@huawei.com>
Wed, 13 Sep 2017 09:52:09 +0000 (09:52 +0000)
few major issues in sli/core/filters module

Issue-Id- CCSDK-87
Change-Id: Idbf2b12b866d48161f089271a2403aaee6b8d04f
Signed-off-by: surya-huawei <a.u.surya@huawei.com>
filters/provider/src/main/java/org/onap/ccsdk/sli/core/filters/LogFilter.java

index 4b14b21..abbb7c6 100644 (file)
@@ -144,7 +144,7 @@ public class LogFilter implements Filter {
             String authzHeader = request.getHeader("Authorization");
             String usernameAndPassword = new String(Base64.decodeBase64(authzHeader.substring(6).getBytes()));
 
-            int userNameIndex = usernameAndPassword.indexOf(":");
+            int userNameIndex = usernameAndPassword.indexOf(':');
             String username = usernameAndPassword.substring(0, userNameIndex);
             userName = username;
 
@@ -171,7 +171,7 @@ public class LogFilter implements Filter {
         MDC.put(END_TIMESTAMP,asIso8601(System.currentTimeMillis()));
         //MDC.put(REQUEST_ID,"already done above");
         MDC.put(SERVICE_NAME,request.getRequestURL().toString());
-        int idx = request.getPathInfo().toString().lastIndexOf(":");
+        int idx = request.getPathInfo().lastIndexOf(':');
         String instance = "";
         if ( idx != -1 ) {
                        instance = request.getPathInfo().substring(idx+1);
@@ -186,13 +186,13 @@ public class LogFilter implements Filter {
                        MDC.put(STATUS_CODE,"COMPLETE");
                }
 
-        MDC.put(RESP_CODE,"" + response.getStatus());
+        MDC.put(RESP_CODE, Integer.toString(response.getStatus()));
         MDC.put(RESP_DESC,"");
         MDC.put(INSTANCE_UUID,"");
         MDC.put(CATEGORY,"");
         MDC.put(SEVERITY,"");
         //MDC.put(SERVER_IP,""); //by chef
-        MDC.put(ELAPSED_TIME,"" + (System.currentTimeMillis() - startTime));
+        MDC.put(ELAPSED_TIME,Long.toString(System.currentTimeMillis() - startTime));
         //MDC.put(SERVER_HOST,""); //by chef
         MDC.put(CLIENT_IP,request.getRemoteHost());
         MDC.put(CLASS,"");