diff options
author | Parshad Patel <pars.patel@samsung.com> | 2019-08-07 16:00:20 +0900 |
---|---|---|
committer | Parshad Patel <pars.patel@samsung.com> | 2019-08-07 19:31:21 +0900 |
commit | fe2fd7021b7cb8a7c2767c1df2e3586b5041018f (patch) | |
tree | ea43935df9ecdd59fa217dc7bcc4746fd6a0210c /ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service | |
parent | 71cf9cbe286d92e38b517e3a5829ba833859c93f (diff) |
Sonar fix in epsdk-core
A "NullPointerException" could be thrown; "appuser" is nullable here
Either re-interrupt this method or rethrow the "InterruptedException"
Issue-ID: PORTAL-562
Change-Id: Idc3cf823d42ad4afc3ca1ae58c3994916e0d8142
Signed-off-by: Parshad Patel <pars.patel@samsung.com>
Diffstat (limited to 'ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service')
-rw-r--r-- | ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service/LoginExternalAuthServiceImpl.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service/LoginExternalAuthServiceImpl.java b/ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service/LoginExternalAuthServiceImpl.java index c31ef75e..d0450966 100644 --- a/ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service/LoginExternalAuthServiceImpl.java +++ b/ecomp-sdk/epsdk-core/src/main/java/org/onap/portalsdk/core/service/LoginExternalAuthServiceImpl.java @@ -126,8 +126,9 @@ public class LoginExternalAuthServiceImpl implements LoginExternalAuthService { if (appuser == null && userHasRoleFunctions(user)) { createUserIfNecessary(user); } else { - appuser.setLastLoginDate(new Date()); - + if (appuser != null) { + appuser.setLastLoginDate(new Date()); + } // update the last logged in date for the user dataAccessService.saveDomainObject(appuser, additionalParams); } |