diff options
author | 2018-09-15 12:23:47 +0530 | |
---|---|---|
committer | 2018-09-15 12:27:04 +0530 | |
commit | 08d41a3fdcf909466e82c43ba30fefa0cf80c41e (patch) | |
tree | 9c6bd3532eb377467467da1308db9eaf24bdd133 /src/main/java | |
parent | 23bdd1ef418e486c7306862dbba8766df11690bc (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>
Diffstat (limited to 'src/main/java')
-rw-r--r-- | src/main/java/org/onap/dmaap/dbcapi/aaf/AafConnection.java | 22 |
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(); |