From 6c7c35c88c3839c667a1f100afc9a650c8cec279 Mon Sep 17 00:00:00 2001
From: nivetha <nivetha.mariappan@tarento.com>
Date: Mon, 28 Feb 2022 11:49:19 +0530
Subject: [PATCH] removed stacktrace

---
 .../com/tarento/retail/controller/RoleActionController.java | 6 +++++-
 src/main/java/com/tarento/retail/dao/impl/UserDaoImpl.java  | 1 -
 .../com/tarento/retail/service/impl/UserServiceImpl.java    | 2 +-
 .../java/com/tarento/retail/util/NotificationService.java   | 1 -
 4 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/tarento/retail/controller/RoleActionController.java b/src/main/java/com/tarento/retail/controller/RoleActionController.java
index 69a0c23..6e9469a 100644
--- a/src/main/java/com/tarento/retail/controller/RoleActionController.java
+++ b/src/main/java/com/tarento/retail/controller/RoleActionController.java
@@ -4,6 +4,8 @@ import java.util.ArrayList;
 import java.util.List;
 
 import org.apache.commons.lang3.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -36,6 +38,8 @@ import com.tarento.retail.util.ResponseMessages;
 @RequestMapping(PathRoutes.USER_ACTIONS_URL)
 public class RoleActionController {
 
+	public static final Logger LOGGER = LoggerFactory.getLogger(RoleActionController.class);
+
 	@Autowired
 	private RoleActionService roleActionService;
 
@@ -66,7 +70,7 @@ public class RoleActionController {
 		try {
 			return ResponseGenerator.successResponse(roleActionService.getAllRolesByDomain(orgDomain));
 		} catch (JsonProcessingException e) {
-			e.printStackTrace();
+			LOGGER.error("Exception in listRolesByDomain :" + e.getMessage());
 		}
 		return ResponseGenerator.failureResponse(ResponseMessages.ErrorMessages.ORG_DOMAIN_CODE_UNAVAILABLE);
 	}
diff --git a/src/main/java/com/tarento/retail/dao/impl/UserDaoImpl.java b/src/main/java/com/tarento/retail/dao/impl/UserDaoImpl.java
index 11ac9e0..b6e4a30 100644
--- a/src/main/java/com/tarento/retail/dao/impl/UserDaoImpl.java
+++ b/src/main/java/com/tarento/retail/dao/impl/UserDaoImpl.java
@@ -398,7 +398,6 @@ public class UserDaoImpl implements UserDao {
 				}
 			}, keyHolder);
 		} catch (Exception e) {
-			e.printStackTrace();
 			LOGGER.error(String.format(Constants.EXCEPTION_METHOD, "saveUserProfile", e.getMessage()));
 			return null;
 		}
diff --git a/src/main/java/com/tarento/retail/service/impl/UserServiceImpl.java b/src/main/java/com/tarento/retail/service/impl/UserServiceImpl.java
index cba7d19..8cf3d4d 100644
--- a/src/main/java/com/tarento/retail/service/impl/UserServiceImpl.java
+++ b/src/main/java/com/tarento/retail/service/impl/UserServiceImpl.java
@@ -313,7 +313,7 @@ public class UserServiceImpl implements UserDetailsService, UserService {
 				return false;
 
 		} catch (IOException e) {
-			e.printStackTrace();
+			LOGGER.error(String.format(Constants.EXCEPTION_METHOD, "uploadFile", e.getMessage()));
 			return false;
 		}
 	}
diff --git a/src/main/java/com/tarento/retail/util/NotificationService.java b/src/main/java/com/tarento/retail/util/NotificationService.java
index eefa88e..3f10909 100644
--- a/src/main/java/com/tarento/retail/util/NotificationService.java
+++ b/src/main/java/com/tarento/retail/util/NotificationService.java
@@ -77,7 +77,6 @@ public class NotificationService {
 			transport.close();
 			return Boolean.TRUE;
 		} catch (Exception e) {
-			e.printStackTrace();
 			LOGGER.error(String.format("Exception in %s : %s", "sendMail", e.getMessage()));
 		}
 		return Boolean.FALSE;
-- 
GitLab