Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[톰캣 구현하기 - 1,2단계] 베로(김은솔) 미션 제출합니다. #325

Merged
merged 29 commits into from
Sep 6, 2023
Merged
Show file tree
Hide file tree
Changes from 20 commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
839ce23
test: 학습 테스트 완료
Cyma-s Sep 2, 2023
cc4330d
feat: index.html 응답하도록 수정
Cyma-s Sep 2, 2023
d9fbe85
feat: CSS, js 파일을 지원하도록 수정
Cyma-s Sep 2, 2023
f9ed141
feat: 로그인 시 user 정보 logging 기능 추가
Cyma-s Sep 2, 2023
b464eb2
feat: 로그인 결과에 따라 적절한 페이지로 리다이렉트 하도록 추가
Cyma-s Sep 3, 2023
3abf0bd
feat: 회원가입 기능 추가
Cyma-s Sep 3, 2023
2da4b10
feat: 쿠키에 JSESSIONID 값 저장하는 기능 추가
Cyma-s Sep 3, 2023
a72d6c4
feat: 유효한 Session이 존재하면 index.html 로 리다이렉트하는 기능 추가
Cyma-s Sep 4, 2023
6ed967f
test: 통합 테스트 추가
Cyma-s Sep 4, 2023
823621a
fix: 헤더 알파벳 순 정렬 추가
Cyma-s Sep 4, 2023
5ee4610
refactor: optional 값 변수로 추출
Cyma-s Sep 4, 2023
da74a18
refactor: stream 에 try-with-resources 적용
Cyma-s Sep 4, 2023
07b61d8
fix: 로깅 삭제
Cyma-s Sep 4, 2023
fa0d6f9
refactor: 생성자 접근 제어자 변경
Cyma-s Sep 4, 2023
5189da0
refactor: 불필요한 출력문 삭제
Cyma-s Sep 4, 2023
1b152ef
refactor: 쿠키 파싱 로직 리팩터링
Cyma-s Sep 4, 2023
5c588a0
refactor: static 클래스 private 생성자 추가
Cyma-s Sep 4, 2023
22379dc
refactor: EnumMap 으로 변경
Cyma-s Sep 4, 2023
9ecd360
refactor: 변수 이름 변경
Cyma-s Sep 4, 2023
48141c7
refactor: assertAll 로 감싸도록 변경
Cyma-s Sep 4, 2023
b354bfa
refactor: HttpMethod가 같은지 확인하는 메서드 추가
Cyma-s Sep 5, 2023
509870e
refactor: 사용하지 않는 생성자 삭제
Cyma-s Sep 5, 2023
74b22bf
refactor: 생성자 접근 제어자를 private 으로 변경
Cyma-s Sep 5, 2023
36ab2de
refactor: 세션 검증 메서드 추가
Cyma-s Sep 5, 2023
8971b54
refactor: HttpRequest 생성 책임 분리
Cyma-s Sep 5, 2023
67e89de
refactor: 빈 favicon 파일 추가
Cyma-s Sep 5, 2023
2e58ab4
refactor: 메서드 이름 변경
Cyma-s Sep 5, 2023
ca77c9e
refactor: 어색하지 않은 변수 이름으로 변경
Cyma-s Sep 5, 2023
5323db0
refactor: 메서드 이름 오타 수정
Cyma-s Sep 5, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 9 additions & 8 deletions study/src/test/java/study/FileTest.java
Original file line number Diff line number Diff line change
@@ -1,11 +1,15 @@
package study;

import java.io.FileNotFoundException;
import java.io.IOException;
import java.nio.file.Files;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;

import java.nio.file.Path;
import java.util.Collections;
import java.util.List;
import org.springframework.util.ResourceUtils;

import static org.assertj.core.api.Assertions.assertThat;

Expand All @@ -24,11 +28,10 @@ class FileTest {
* resource 디렉터리의 경로는 어떻게 알아낼 수 있을까?
*/
@Test
void resource_디렉터리에_있는_파일의_경로를_찾는다() {
void resource_디렉터리에_있는_파일의_경로를_찾는다() throws IOException {
final String fileName = "nextstep.txt";

// todo
final String actual = "";
final String actual = ResourceUtils.getURL("classpath:" + fileName).getPath();

assertThat(actual).endsWith(fileName);
}
Expand All @@ -40,14 +43,12 @@ class FileTest {
* File, Files 클래스를 사용하여 파일의 내용을 읽어보자.
*/
@Test
void 파일의_내용을_읽는다() {
void 파일의_내용을_읽는다() throws IOException {
final String fileName = "nextstep.txt";

// todo
final Path path = null;
final Path path = ResourceUtils.getFile("classpath:" + fileName).toPath();

// todo
final List<String> actual = Collections.emptyList();
final List<String> actual = Files.readAllLines(path);

assertThat(actual).containsOnly("nextstep");
}
Expand Down
133 changes: 68 additions & 65 deletions study/src/test/java/study/IOStreamTest.java
Original file line number Diff line number Diff line change
@@ -1,45 +1,52 @@
package study;

import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.atLeastOnce;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.BufferedReader;
import java.io.ByteArrayInputStream;
import java.io.ByteArrayOutputStream;
import java.io.FilterInputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.OutputStream;
import java.nio.charset.StandardCharsets;
import java.util.stream.Collectors;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Nested;
import org.junit.jupiter.api.Test;

import java.io.*;

import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.*;

/**
* 자바는 스트림(Stream)으로부터 I/O를 사용한다.
* 입출력(I/O)은 하나의 시스템에서 다른 시스템으로 데이터를 이동 시킬 때 사용한다.
*
* InputStream은 데이터를 읽고, OutputStream은 데이터를 쓴다.
* FilterStream은 InputStream이나 OutputStream에 연결될 수 있다.
* FilterStream은 읽거나 쓰는 데이터를 수정할 때 사용한다. (e.g. 암호화, 압축, 포맷 변환)
*
* Stream은 데이터를 바이트로 읽고 쓴다.
* 바이트가 아닌 텍스트(문자)를 읽고 쓰려면 Reader와 Writer 클래스를 연결한다.
* Reader, Writer는 다양한 문자 인코딩(e.g. UTF-8)을 처리할 수 있다.
* 자바는 스트림(Stream)으로부터 I/O를 사용한다. 입출력(I/O)은 하나의 시스템에서 다른 시스템으로 데이터를 이동 시킬 때 사용한다.
* <p>
* InputStream은 데이터를 읽고, OutputStream은 데이터를 쓴다. FilterStream은 InputStream이나 OutputStream에 연결될 수 있다. FilterStream은 읽거나 쓰는
* 데이터를 수정할 때 사용한다. (e.g. 암호화, 압축, 포맷 변환)
* <p>
* Stream은 데이터를 바이트로 읽고 쓴다. 바이트가 아닌 텍스트(문자)를 읽고 쓰려면 Reader와 Writer 클래스를 연결한다. Reader, Writer는 다양한 문자 인코딩(e.g. UTF-8)을
* 처리할 수 있다.
*/
@DisplayName("Java I/O Stream 클래스 학습 테스트")
class IOStreamTest {

/**
* OutputStream 학습하기
*
* 자바의 기본 출력 클래스는 java.io.OutputStream이다.
* OutputStream의 write(int b) 메서드는 기반 메서드이다.
* <p>
* 자바의 기본 출력 클래스는 java.io.OutputStream이다. OutputStream의 write(int b) 메서드는 기반 메서드이다.
* <code>public abstract void write(int b) throws IOException;</code>
*/
@Nested
class OutputStream_학습_테스트 {

/**
* OutputStream은 다른 매체에 바이트로 데이터를 쓸 때 사용한다.
* OutputStream의 서브 클래스(subclass)는 특정 매체에 데이터를 쓰기 위해 write(int b) 메서드를 사용한다.
* 예를 들어, FilterOutputStream은 파일로 데이터를 쓸 때,
* 또는 DataOutputStream은 자바의 primitive type data를 다른 매체로 데이터를 쓸 때 사용한다.
*
* OutputStream은 다른 매체에 바이트로 데이터를 쓸 때 사용한다. OutputStream의 서브 클래스(subclass)는 특정 매체에 데이터를 쓰기 위해 write(int b) 메서드를
* 사용한다. 예를 들어, FilterOutputStream은 파일로 데이터를 쓸 때, 또는 DataOutputStream은 자바의 primitive type data를 다른 매체로 데이터를 쓸 때
* 사용한다.
* <p>
* write 메서드는 데이터를 바이트로 출력하기 때문에 비효율적이다.
* <code>write(byte[] data)</code>와 <code>write(byte b[], int off, int len)</code> 메서드는
* 1바이트 이상을 한 번에 전송 할 수 있어 훨씬 효율적이다.
Expand All @@ -53,6 +60,7 @@ class OutputStream_학습_테스트 {
* todo
* OutputStream 객체의 write 메서드를 사용해서 테스트를 통과시킨다
*/
outputStream.write("nextstep".getBytes());

final String actual = outputStream.toString();

Expand All @@ -61,13 +69,10 @@ class OutputStream_학습_테스트 {
}

/**
* 효율적인 전송을 위해 스트림에서 버퍼링을 사용 할 수 있다.
* BufferedOutputStream 필터를 연결하면 버퍼링이 가능하다.
*
* 버퍼링을 사용하면 OutputStream을 사용할 때 flush를 사용하자.
* flush() 메서드는 버퍼가 아직 가득 차지 않은 상황에서 강제로 버퍼의 내용을 전송한다.
* Stream은 동기(synchronous)로 동작하기 때문에 버퍼가 찰 때까지 기다리면
* 데드락(deadlock) 상태가 되기 때문에 flush로 해제해야 한다.
* 효율적인 전송을 위해 스트림에서 버퍼링을 사용 할 수 있다. BufferedOutputStream 필터를 연결하면 버퍼링이 가능하다.
* <p>
* 버퍼링을 사용하면 OutputStream을 사용할 때 flush를 사용하자. flush() 메서드는 버퍼가 아직 가득 차지 않은 상황에서 강제로 버퍼의 내용을 전송한다. Stream은
* 동기(synchronous)로 동작하기 때문에 버퍼가 찰 때까지 기다리면 데드락(deadlock) 상태가 되기 때문에 flush로 해제해야 한다.
*/
@Test
void BufferedOutputStream을_사용하면_버퍼링이_가능하다() throws IOException {
Expand All @@ -78,14 +83,14 @@ class OutputStream_학습_테스트 {
* flush를 사용해서 테스트를 통과시킨다.
* ByteArrayOutputStream과 어떤 차이가 있을까?
*/
outputStream.flush();

verify(outputStream, atLeastOnce()).flush();
outputStream.close();
}

/**
* 스트림 사용이 끝나면 항상 close() 메서드를 호출하여 스트림을 닫는다.
* 장시간 스트림을 닫지 않으면 파일, 포트 등 다양한 리소스에서 누수(leak)가 발생한다.
* 스트림 사용이 끝나면 항상 close() 메서드를 호출하여 스트림을 닫는다. 장시간 스트림을 닫지 않으면 파일, 포트 등 다양한 리소스에서 누수(leak)가 발생한다.
*/
@Test
void OutputStream은_사용하고_나서_close_처리를_해준다() throws IOException {
Expand All @@ -96,27 +101,26 @@ class OutputStream_학습_테스트 {
* try-with-resources를 사용한다.
* java 9 이상에서는 변수를 try-with-resources로 처리할 수 있다.
*/
outputStream.close();

verify(outputStream, atLeastOnce()).close();
}
}

/**
* InputStream 학습하기
*
* 자바의 기본 입력 클래스는 java.io.InputStream이다.
* InputStream은 다른 매체로부터 바이트로 데이터를 읽을 때 사용한다.
* InputStream의 read() 메서드는 기반 메서드이다.
* <p>
* 자바의 기본 입력 클래스는 java.io.InputStream이다. InputStream은 다른 매체로부터 바이트로 데이터를 읽을 때 사용한다. InputStream의 read() 메서드는 기반
* 메서드이다.
* <code>public abstract int read() throws IOException;</code>
*
* <p>
* InputStream의 서브 클래스(subclass)는 특정 매체에 데이터를 읽기 위해 read() 메서드를 사용한다.
*/
@Nested
class InputStream_학습_테스트 {

/**
* read() 메서드는 매체로부터 단일 바이트를 읽는데, 0부터 255 사이의 값을 int 타입으로 반환한다.
* int 값을 byte 타입으로 변환하면 -128부터 127 사이의 값으로 변환된다.
* read() 메서드는 매체로부터 단일 바이트를 읽는데, 0부터 255 사이의 값을 int 타입으로 반환한다. int 값을 byte 타입으로 변환하면 -128부터 127 사이의 값으로 변환된다.
* 그리고 Stream 끝에 도달하면 -1을 반환한다.
*/
@Test
Expand All @@ -128,16 +132,15 @@ class InputStream_학습_테스트 {
* todo
* inputStream에서 바이트로 반환한 값을 문자열로 어떻게 바꿀까?
*/
final String actual = "";
final String actual = new BufferedReader(new InputStreamReader(inputStream)).readLine();

assertThat(actual).isEqualTo("🤩");
assertThat(inputStream.read()).isEqualTo(-1);
inputStream.close();
}

/**
* 스트림 사용이 끝나면 항상 close() 메서드를 호출하여 스트림을 닫는다.
* 장시간 스트림을 닫지 않으면 파일, 포트 등 다양한 리소스에서 누수(leak)가 발생한다.
* 스트림 사용이 끝나면 항상 close() 메서드를 호출하여 스트림을 닫는다. 장시간 스트림을 닫지 않으면 파일, 포트 등 다양한 리소스에서 누수(leak)가 발생한다.
*/
@Test
void InputStream은_사용하고_나서_close_처리를_해준다() throws IOException {
Expand All @@ -148,64 +151,64 @@ class InputStream_학습_테스트 {
* try-with-resources를 사용한다.
* java 9 이상에서는 변수를 try-with-resources로 처리할 수 있다.
*/
inputStream.close();

verify(inputStream, atLeastOnce()).close();
}
}

/**
* FilterStream 학습하기
*
* 필터는 필터 스트림, reader, writer로 나뉜다.
* 필터는 바이트를 다른 데이터 형식으로 변환 할 때 사용한다.
* reader, writer는 UTF-8, ISO 8859-1 같은 형식으로 인코딩된 텍스트를 처리하는 데 사용된다.
* <p>
* 필터는 필터 스트림, reader, writer로 나뉜다. 필터는 바이트를 다른 데이터 형식으로 변환 할 때 사용한다. reader, writer는 UTF-8, ISO 8859-1 같은 형식으로 인코딩된
* 텍스트를 처리하는 데 사용된다.
*/
@Nested
class FilterStream_학습_테스트 {

/**
* BufferedInputStream은 데이터 처리 속도를 높이기 위해 데이터를 버퍼에 저장한다.
* InputStream 객체를 생성하고 필터 생성자에 전달하면 필터에 연결된다.
* 버퍼 크기를 지정하지 않으면 버퍼의 기본 사이즈는 얼마일까?
* BufferedInputStream은 데이터 처리 속도를 높이기 위해 데이터를 버퍼에 저장한다. InputStream 객체를 생성하고 필터 생성자에 전달하면 필터에 연결된다. 버퍼 크기를 지정하지
* 않으면 버퍼의 기본 사이즈는 얼마일까? -> 8192 byte
*/
@Test
void 필터인_BufferedInputStream를_사용해보자() {
final String text = "필터에 연결해보자.";
final InputStream inputStream = new ByteArrayInputStream(text.getBytes());
final InputStream bufferedInputStream = null;
try (final InputStream bufferedInputStream = new BufferedInputStream(inputStream)) {
final byte[] actual = bufferedInputStream.readAllBytes();

final byte[] actual = new byte[0];
assertThat(bufferedInputStream).isInstanceOf(FilterInputStream.class);
assertThat(actual).isEqualTo("필터에 연결해보자.".getBytes());

assertThat(bufferedInputStream).isInstanceOf(FilterInputStream.class);
assertThat(actual).isEqualTo("필터에 연결해보자.".getBytes());
} catch (IOException e) {
throw new RuntimeException(e);
}
}
}

/**
* 자바의 기본 문자열은 UTF-16 유니코드 인코딩을 사용한다.
* 문자열이 아닌 바이트 단위로 처리하려니 불편하다.
* 그리고 바이트를 문자(char)로 처리하려면 인코딩을 신경 써야 한다.
* reader, writer를 사용하면 입출력 스트림을 바이트가 아닌 문자 단위로 데이터를 처리하게 된다.
* 그리고 InputStreamReader를 사용하면 지정된 인코딩에 따라 유니코드 문자로 변환할 수 있다.
* 자바의 기본 문자열은 UTF-16 유니코드 인코딩을 사용한다. 문자열이 아닌 바이트 단위로 처리하려니 불편하다. 그리고 바이트를 문자(char)로 처리하려면 인코딩을 신경 써야 한다. reader,
* writer를 사용하면 입출력 스트림을 바이트가 아닌 문자 단위로 데이터를 처리하게 된다. 그리고 InputStreamReader를 사용하면 지정된 인코딩에 따라 유니코드 문자로 변환할 수 있다.
*/
@Nested
class InputStreamReader_학습_테스트 {

/**
* InputStreamReader를 사용해서 바이트를 문자(char)로 읽어온다.
* 읽어온 문자(char)를 문자열(String)로 처리하자.
* 필터인 BufferedReader를 사용하면 readLine 메서드를 사용해서 문자열(String)을 한 줄 씩 읽어올 수 있다.
* InputStreamReader를 사용해서 바이트를 문자(char)로 읽어온다. 읽어온 문자(char)를 문자열(String)로 처리하자. 필터인 BufferedReader를 사용하면
* readLine 메서드를 사용해서 문자열(String)을 한 줄 씩 읽어올 수 있다.
*/
@Test
void BufferedReader를_사용하여_문자열을_읽어온다() {
final String emoji = String.join("\r\n",
"😀😃😄😁😆😅😂🤣🥲☺️😊",
"😇🙂🙃😉😌😍🥰😘😗😙😚",
"😋😛😝😜🤪🤨🧐🤓😎🥸🤩",
"");
"😀😃😄😁😆😅😂🤣🥲☺️😊",
"😇🙂🙃😉😌😍🥰😘😗😙😚",
"😋😛😝😜🤪🤨🧐🤓😎🥸🤩",
"");
final InputStream inputStream = new ByteArrayInputStream(emoji.getBytes());

final StringBuilder actual = new StringBuilder();
new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8)).lines()
.forEach(line -> actual.append(line).append("\r\n"));

assertThat(actual).hasToString(emoji);
}
Expand Down
11 changes: 11 additions & 0 deletions tomcat/src/main/java/nextstep/filter/Interceptor.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
package nextstep.filter;

import org.apache.coyote.http11.HttpRequest;
import org.apache.coyote.http11.HttpResponse;

public interface Interceptor {

boolean preHandle(final HttpRequest httpRequest, final HttpResponse httpResponse);

boolean support(final HttpRequest httpRequest);
}
33 changes: 33 additions & 0 deletions tomcat/src/main/java/nextstep/filter/LoginInterceptor.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
package nextstep.filter;

import static org.apache.coyote.http11.StaticPages.INDEX_PAGE;

import java.util.List;
import nextstep.jwp.exception.InvalidSessionException;
import org.apache.catalina.SessionManager;
import org.apache.coyote.http11.HttpRequest;
import org.apache.coyote.http11.HttpResponse;
import org.apache.coyote.http11.Session;

public class LoginInterceptor implements Interceptor {

private static final List<String> SUPPORT_PATH = List.of(
"/login"
);

@Override
public boolean support(final HttpRequest httpRequest) {
return SUPPORT_PATH.contains(httpRequest.getPath());
}

@Override
public boolean preHandle(final HttpRequest httpRequest, final HttpResponse httpResponse) {
if (httpRequest.containsCookieAndJSessionID()) {
final Session session = SessionManager.getInstance().findSession(httpRequest.getCookie().getJSessionID())

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

session 이 사용되지 않고 있는데 혹시 Exception 처리 때문에 사용하신걸까요??

현재에는 별다른 로직이 없어서 사용되지 않는 것 같긴합니다만,,
현재 요구사항에 맞는 메서드를 하나 추가하시는건 어떠실까요?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exception 처리 때문에 사용한 게 맞습니다 예리하시군요
그리고 sonar cloud 에서 orElseThrow 를 할 때는 변수를 만들어야 한다고 경고해서 변수를 만들어주도록 했습니다.
우르 말대로 요구사항에 맞는 메서드가 있는 게 좋을 것 같아서 validateSession 이라는 메서드를 추가해서 사용했습니다 !

.orElseThrow(InvalidSessionException::new);
httpResponse.sendRedirect(INDEX_PAGE);
return false;
}
return true;
}
}
11 changes: 11 additions & 0 deletions tomcat/src/main/java/nextstep/jwp/controller/Controller.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
package nextstep.jwp.controller;

import java.io.IOException;
import org.apache.coyote.http11.HttpRequest;

public interface Controller {

ResponseEntity service(final HttpRequest httpRequest) throws IOException;

boolean canHandle(final HttpRequest httpRequest);
}
Loading