[1-2단계 톰캣 구현하기] 비토(오상훈) 미션 제출합니다. #526
Conversation
| String fileName = "static" + path; | ||
| var resourceUrl = getClass().getClassLoader().getResource(fileName); | ||
| Path filePath = Path.of(resourceUrl.toURI()); |
There was a problem hiding this comment.
존재하지 않는 경로에 대한 처리가 없어서 NullPointerException 발생합니다.
| String requestBody = httpRequest.getBody(); | ||
| String[] token = requestBody.split("&"); | ||
| String account = token[0].split("=")[1]; | ||
| String password = token[1].split("=")[1]; |
There was a problem hiding this comment.
로그인 페이지에서 input 값을 완전히 채우지 않고 POST 요청을 보내면 ArrayIndexOutOfBoundsException 예외가 발생합니다.
| String requestBody = httpRequest.getBody(); | ||
| String[] token = requestBody.split("&"); | ||
| String account = token[0].split("=")[1]; | ||
| String email = token[1].split("=")[1]; | ||
| String password = token[2].split("=")[1]; |
There was a problem hiding this comment.
회원가입 페이지에서 input 값을 완전히 채우지 않고 POST 요청을 보내면 ArrayIndexOutOfBoundsException 예외가 발생합니다.
| User user = new User(account, password, email); | ||
| InMemoryUserRepository.save(user); |
| public RequestMapping() { | ||
| controllers.put("/login", new LoginController()); | ||
| controllers.put("/register", new RegisterController()); | ||
| controllers.put("page", new PageController()); |
There was a problem hiding this comment.
로그인, 회원가입 페이지가 아니면 PageController를 반환하고 있어서 controllers에 page를 넣을 필요가 없을 것 같네요.
| Map<String, String> headers = httpResponseHeader.getHeaders(); | ||
| StringBuilder sb = new StringBuilder(); | ||
| int size = headers.keySet().size(); | ||
| int i = 1; | ||
| for (String key : headers.keySet()) { | ||
| if (i < size) { | ||
| sb.append(key).append(": ").append(headers.get(key)).append(" \r\n"); | ||
| size++; | ||
| } else { | ||
| sb.append(key).append(": ").append(headers.get(key)); | ||
| } | ||
| } |
There was a problem hiding this comment.
HttpResponseHeader 객체에게 responseHeader를 요청하는게 어떨까요?
| String[] cookies = httpRequest.getValue("Cookie").split("; "); | ||
| String cookie = ""; | ||
| for (String c : cookies) { | ||
| if (c.contains("JSESSIONID")) { | ||
| cookie = c.split("=")[1]; | ||
| } | ||
| } |
There was a problem hiding this comment.
어느 미친 사용자가 비토가 넣어준 JSESSIONID 값을 지우고 로그인 페이지로 접근했더니ArrayIndexOutOfBoundsException예외가 발생했다네요.
| String path = httpRequest.getPath(); | ||
| if (!httpRequest.getPath().contains(".")) { | ||
| path += ".html"; | ||
| } |
There was a problem hiding this comment.
어느 미친 사용자가 서버 내부 에러를 발생시켰는데, 그 요청은 localhost:8080/500 이었어요.
| ; | ||
|
|
||
| private final String contentType; | ||
| private final String extention; |
안녕하세요 이상! 비토입니다!
어느 정도 클래스 분리를 하긴했는데 아직 군데군데 리팩토링이 필요하긴 합니다.
천천히 리뷰 주세요.
리뷰 잘 부탁드립니다!