diff options
author | r.bogacki <r.bogacki@samsung.com> | 2019-03-19 14:27:16 +0100 |
---|---|---|
committer | Robert Bogacki <r.bogacki@samsung.com> | 2019-03-28 06:56:23 +0000 |
commit | 0a358ea9fd56796d7dcf9c2a50df82a4f05b9738 (patch) | |
tree | 8236f863b96b429e109d398d992782b48c803104 /src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java | |
parent | cf8096a9fe50de511d5864032ed4825a2a314fa5 (diff) |
Log stack traces with logger
Fixed critical sonar issues:
-"Either log or rethrow this exception"
Change-Id: I99a550f2cf40bd6c34a0f8ca1893c964669c76de
Issue-ID: MUSIC-367
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Diffstat (limited to 'src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java')
-rw-r--r-- | src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java b/src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java index 2d0d4e59..0fd6c0e2 100644 --- a/src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java +++ b/src/main/java/org/onap/music/authentication/MusicAAFAuthentication.java @@ -4,6 +4,8 @@ * =================================================================== * Copyright (c) 2017 AT&T Intellectual Property * =================================================================== + * Modifications Copyright (c) 2019 Samsung + * =================================================================== * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at @@ -134,7 +136,7 @@ public class MusicAAFAuthentication implements MusicAuthenticator { isValid = CachingUtil.authenticateAAFUser(namespace, userId, password, keyspace); } catch (Exception e) { logger.error(EELFLoggerDelegate.errorLogger,"Error while aaf authentication for user:" + userId); - logger.error(EELFLoggerDelegate.errorLogger,"Error: "+ e.getMessage()); + logger.error(EELFLoggerDelegate.errorLogger,"Error: "+ e.getMessage(), e); logger.error(EELFLoggerDelegate.errorLogger,e.getMessage(), AppMessages.AUTHENTICATIONERROR ,ErrorSeverity.WARN, ErrorTypes.AUTHENTICATIONERROR); logger.error(EELFLoggerDelegate.errorLogger,"Got exception while AAF authentication for namespace " + namespace); resultMap.put("Exception", e.getMessage()); @@ -170,7 +172,7 @@ public class MusicAAFAuthentication implements MusicAuthenticator { uuid = rs.getUUID("uuid").toString(); resultMap.put("uuid", "existing"); } catch (Exception e) { - logger.error(EELFLoggerDelegate.applicationLogger,"No UUID found in DB. So creating new UUID."); + logger.error(EELFLoggerDelegate.applicationLogger,"No UUID found in DB. So creating new UUID.", e); uuid = MusicUtil.generateUUID(); resultMap.put("uuid", "new"); } |