aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAmaresh Kumar <kamaresh@in.ibm.com>2018-09-15 12:23:47 +0530
committerAmaresh Kumar <kamaresh@in.ibm.com>2018-09-15 12:27:04 +0530
commit08d41a3fdcf909466e82c43ba30fefa0cf80c41e (patch)
tree9c6bd3532eb377467467da1308db9eaf24bdd133
parent23bdd1ef418e486c7306862dbba8766df11690bc (diff)
critical sonar fixed in AafConnection.java
https://sonar.onap.org/project/issues?fileUuids=AWBPi64T-08if2a6vO68&id=org.onap.dmaap.dbcapi%3Adbcapi&open=AWBPi68C-08if2a6vO8k&resolved=false&severities=CRITICAL Issue-ID: DMAAP-779 Change-Id: I540bf00862ef643a1319a5dee969f786216da100 Signed-off-by: Amaresh Kumar <kamaresh@in.ibm.com>
-rw-r--r--src/main/java/org/onap/dmaap/dbcapi/aaf/AafConnection.java22
1 files changed, 11 insertions, 11 deletions
diff --git a/src/main/java/org/onap/dmaap/dbcapi/aaf/AafConnection.java b/src/main/java/org/onap/dmaap/dbcapi/aaf/AafConnection.java
index b1d3871..1d7b273 100644
--- a/src/main/java/org/onap/dmaap/dbcapi/aaf/AafConnection.java
+++ b/src/main/java/org/onap/dmaap/dbcapi/aaf/AafConnection.java
@@ -46,13 +46,13 @@ import org.onap.dmaap.dbcapi.util.DmaapConfig;
public class AafConnection extends BaseLoggingClass {
-
-
+
+
private String aafCred;
private String unit_test;
-
+
private HttpsURLConnection uc;
@@ -62,10 +62,10 @@ public class AafConnection extends BaseLoggingClass {
unit_test = p.getProperty( "UnitTest", "No" );
}
-
+
private boolean makeConnection( String pURL ) {
-
+
try {
URL u = new URL( pURL );
uc = (HttpsURLConnection) u.openConnection();
@@ -78,7 +78,7 @@ public class AafConnection extends BaseLoggingClass {
return(false);
} catch (Exception e) {
logger.error("Error", e);
- errorLogger.error(DmaapbcLogMessageEnum.HTTP_CONNECTION_ERROR, pURL, e.getMessage() );
+ errorLogger.error(DmaapbcLogMessageEnum.HTTP_CONNECTION_ERROR, pURL, e.getMessage());
e.printStackTrace();
return(false);
}
@@ -94,9 +94,9 @@ public class AafConnection extends BaseLoggingClass {
sb.append( line );
}
} catch (IOException ex ) {
- errorLogger.error( DmaapbcLogMessageEnum.IO_EXCEPTION, ex.getMessage());
+ errorLogger.error( DmaapbcLogMessageEnum.IO_EXCEPTION + ex.getMessage(),ex);
}
-
+
return sb.toString();
}
@@ -107,11 +107,11 @@ public class AafConnection extends BaseLoggingClass {
String auth = "Basic " + Base64.encodeBase64String(aafCred.getBytes());
int rc = -1;
-
+
if ( ! makeConnection( pURL ) ) {
return rc;
};
-
+
byte[] postData = obj.getBytes();
//logger.info( "post fields=" + postData ); //byte isn't very readable
@@ -255,7 +255,7 @@ public class AafConnection extends BaseLoggingClass {
logger.error("Error", e);
}
} catch ( SSLHandshakeException she ) {
- errorLogger.error( DmaapbcLogMessageEnum.SSL_HANDSHAKE_ERROR, pURL);
+ errorLogger.error( DmaapbcLogMessageEnum.SSL_HANDSHAKE_ERROR +"For:- "+pURL,she);
}
try {
rc = uc.getResponseCode();