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

[Jay] 5장 리펙토링 #15

Open
wants to merge 4 commits into
base: jay
Choose a base branch
from
Open

[Jay] 5장 리펙토링 #15

wants to merge 4 commits into from

Conversation

jinan159
Copy link
Collaborator

@jinan159 jinan159 commented Aug 7, 2022

Description

리펙토링

Commits

구현 사항 요약

  1. HttpRequestMessage 클래스에 있던 CONTENT_LENGTH 헤더를 HttpHeader 클래스로 이동
  2. HttpRequestMessage 에서 Http Request Line 파싱 로직을 HttpRequestLine 클래스로 분리
  3. RequestHandler 에서 HttpResponse 출력 로직을 HttpRequestMessage 클래스로 분리

@jinan159 jinan159 self-assigned this Aug 7, 2022
@jinan159 jinan159 changed the title 5장 리펙토링 [Jay] 5장 리펙토링 Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant