BAEL-4019: Added logger to log exceptions

This commit is contained in:
sampadawagde 2020-07-03 19:43:33 +05:30
parent bd617e9cae
commit 6c6dc6b159
2 changed files with 10 additions and 2 deletions

View File

@ -19,9 +19,13 @@ import org.apache.shiro.authz.AuthorizationInfo;
import org.apache.shiro.authz.SimpleAuthorizationInfo; import org.apache.shiro.authz.SimpleAuthorizationInfo;
import org.apache.shiro.realm.jdbc.JdbcRealm; import org.apache.shiro.realm.jdbc.JdbcRealm;
import org.apache.shiro.subject.PrincipalCollection; import org.apache.shiro.subject.PrincipalCollection;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
public class CustomRealm extends JdbcRealm { public class CustomRealm extends JdbcRealm {
private Logger logger = LoggerFactory.getLogger(CustomRealm.class);
private Map<String, String> credentials = new HashMap<>(); private Map<String, String> credentials = new HashMap<>();
private Map<String, Set<String>> roles = new HashMap<>(); private Map<String, Set<String>> roles = new HashMap<>();
private Map<String, Set<String>> permissions = new HashMap<>(); private Map<String, Set<String>> permissions = new HashMap<>();
@ -60,7 +64,7 @@ public class CustomRealm extends JdbcRealm {
roles.addAll(getRoleNamesForUser(null, (String) user)); roles.addAll(getRoleNamesForUser(null, (String) user));
permissions.addAll(getPermissions(null, null, roles)); permissions.addAll(getPermissions(null, null, roles));
} catch (SQLException e) { } catch (SQLException e) {
e.printStackTrace(); logger.error(e.getMessage());
} }
} }
SimpleAuthorizationInfo authInfo = new SimpleAuthorizationInfo(roles); SimpleAuthorizationInfo authInfo = new SimpleAuthorizationInfo(roles);

View File

@ -6,6 +6,8 @@ import org.apache.shiro.SecurityUtils;
import org.apache.shiro.authc.AuthenticationException; import org.apache.shiro.authc.AuthenticationException;
import org.apache.shiro.authc.UsernamePasswordToken; import org.apache.shiro.authc.UsernamePasswordToken;
import org.apache.shiro.subject.Subject; import org.apache.shiro.subject.Subject;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.stereotype.Controller; import org.springframework.stereotype.Controller;
import org.springframework.ui.Model; import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.GetMapping; import org.springframework.web.bind.annotation.GetMapping;
@ -17,6 +19,8 @@ import com.baeldung.shiro.models.UserCredentials;
@Controller @Controller
public class ShiroController { public class ShiroController {
private Logger logger = LoggerFactory.getLogger(ShiroController.class);
@GetMapping("/") @GetMapping("/")
public String index() { public String index() {
return "index"; return "index";
@ -37,7 +41,7 @@ public class ShiroController {
try { try {
subject.login(token); subject.login(token);
} catch (AuthenticationException ae) { } catch (AuthenticationException ae) {
ae.printStackTrace(); logger.error(ae.getMessage());
attr.addFlashAttribute("error", "Invalid Credentials"); attr.addFlashAttribute("error", "Invalid Credentials");
return "redirect:/login"; return "redirect:/login";
} }