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.

Sign up for GitHub

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

Jump to bottom

[nathan] Step 5 : 서블릿과 유사한 구조로 리팩토링 제출 #13

Open
wants to merge 17 commits into
base: nathan
Choose a base branch
from
Open

[nathan] Step 5 : 서블릿과 유사한 구조로 리팩토링 제출 #13

wants to merge 17 commits into from

Conversation

nathan29849
Copy link
Member

Description

  • 서블릿과 유사한 구조로 리팩토링

Commits

  • 공통으로 사용하는 메서드 추출하여 추상 클래스로 변경(Controller)
  • Request Handler의 로직을 Controller로 분산
  • 관련 테스트 코드 수정

nathan29849 and others added 16 commits July 27, 2022 11:05
@nathan29849
Update feature_request.md
da100b1
@nathan29849
refactor: [#12] HttpRequestHeaders Test 및 리팩토링
a71d643
@nathan29849
refactor: [#12] HttpRequestMessageBody Test
0641a75
@nathan29849
feat: [#12] HttpRequestTest 추가
ce1c7bc
@nathan29849
Merge pull request #15 from nathan29849/feature/#12/http-request-test
1854c84 
Feature/#12/http request test
@nathan29849
feat: HttpResponse Test 추가
523b4b1 
related to: #13
@nathan29849
Merge pull request #16 from nathan29849/feature/#13/http-response-test
8bd7631 
feat: HttpResponse Test 추가
@nathan29849
refactor: MimeType을 Enum 클래스로 분리
86e511a 
related to : #14
@nathan29849
refactor: 유저 리스트 버튼 url 변경 (/user/list.html -> /user/list)
8803fae 
related to: #14
@nathan29849
chore: controller 패키지 .gitignore에서 제거(작업하기 위함)
79a457e
@nathan29849
refactor: HttpResponse 메서드 내에서 flush 하도록 변경
da797f9 
related to : #14
@nathan29849
chore: import 정리
8bc073b 
related to : #14
@nathan29849
refactor: RequestHandler를 Controller로 분리 및 추상화 진행
4ff70be 
related to : #14
@nathan29849
refactor: HttpRequest 리팩토링
1718de6 
related to : #14
@nathan29849
refactor: HomeController의 url 받는 부분 수정
e3a6d49 
related to: #14
@nathan29849
add: embedded tomcat 의존성 추가 및 활용
ee6a6b9 
related to : #14
@nathan29849 nathan29849 self-assigned this Aug 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nathan29849 nathan29849

Labels
🕹 제출
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nathan29849