diff --git a/src/main/java/com/example/oneul/domain/user/api/UserApi.java b/src/main/java/com/example/oneul/domain/user/api/UserApi.java index 67732fc..017dca5 100644 --- a/src/main/java/com/example/oneul/domain/user/api/UserApi.java +++ b/src/main/java/com/example/oneul/domain/user/api/UserApi.java @@ -18,6 +18,7 @@ import org.springframework.web.bind.annotation.RestController; + @RestController @RequestMapping(value = "/user") public class UserApi { @@ -48,4 +49,9 @@ public class UserApi { return "logout"; } + @RequestMapping(value="", method=RequestMethod.GET) + public String hello() { + return "hello"; + } + } diff --git a/src/main/java/com/example/oneul/domain/user/service/UserServiceImpl.java b/src/main/java/com/example/oneul/domain/user/service/UserServiceImpl.java index a9a039a..7cfed21 100644 --- a/src/main/java/com/example/oneul/domain/user/service/UserServiceImpl.java +++ b/src/main/java/com/example/oneul/domain/user/service/UserServiceImpl.java @@ -48,7 +48,6 @@ public class UserServiceImpl implements UserService { } else { throw new WrongUsernameAndPasswordException("wrong passowrd"); } - return user; } diff --git a/src/main/java/com/example/oneul/global/util/LoginCheckInterceptor.java b/src/main/java/com/example/oneul/global/util/LoginCheckInterceptor.java index 97adbad..9168f2c 100644 --- a/src/main/java/com/example/oneul/global/util/LoginCheckInterceptor.java +++ b/src/main/java/com/example/oneul/global/util/LoginCheckInterceptor.java @@ -6,6 +6,8 @@ import javax.servlet.http.HttpSession; import com.example.oneul.domain.user.domain.UserEntity; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import org.springframework.beans.factory.annotation.Value; import org.springframework.web.servlet.HandlerInterceptor; @@ -13,10 +15,13 @@ public class LoginCheckInterceptor implements HandlerInterceptor{ @Value("${login-page}") private String loginPage; + private final Logger log = LoggerFactory.getLogger(LoginCheckInterceptor.class); @Override public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception { + log.info("login check prehandler"); HttpSession httpSession = request.getSession(false); if(httpSession == null){ + log.info("no session"); response.sendRedirect(loginPage); return false; } @@ -24,6 +29,7 @@ public class LoginCheckInterceptor implements HandlerInterceptor{ UserEntity userEntity = (UserEntity) httpSession.getAttribute("user"); if(userEntity == null){ + log.info("session expired"); response.sendRedirect(loginPage); return false; } diff --git a/src/test/java/com/example/oneul/controller/UserControllerTest.java b/src/test/java/com/example/oneul/controller/UserControllerTest.java index 1f087ca..2d20e8b 100644 --- a/src/test/java/com/example/oneul/controller/UserControllerTest.java +++ b/src/test/java/com/example/oneul/controller/UserControllerTest.java @@ -124,4 +124,10 @@ public class UserControllerTest { actions.andExpectAll(status().isNotFound()); } + + @Test + @DisplayName("logout test") + public void logoutTest() throws Exception { + + } }