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

[테리] Step 3,4: HTTP 웹서버 구현 #7

Open
wants to merge 10 commits into
base: mybloom
Choose a base branch
from
Open

[테리] Step 3,4: HTTP 웹서버 구현 #7

wants to merge 10 commits into from

Conversation

mybloom
Copy link
Collaborator

@mybloom mybloom commented Jul 23, 2022 •
edited
Loading

Description

Todo

  • 요구사항 0 : CI/CD 구축, Heroku 배포
  • 요구사항 1 : index.html 응답하기
  • 요구사항 2 : GET 방식으로 회원가입하기
  • 요구사항 3 : POST 방식으로 회원가입하기
  • 요구사항 4 : 302 status code 적용
  • 요구사항 5 : 로그인하기
  • 요구사항 6 : 사용자 목록 출력하기
  • 요구사항 7 : CSS 지원하기

mybloom added 10 commits July 22, 2022 10:04
@mybloom
feat: 요구사항1. index.html 응답하기 구현 (#1)
cf6612b 
- http://localhost:8080/index.html 요청의 header 값을 읽는다.
- /webapp/index.html 파일을 읽어 response body를 만든다.
- 응답 헤더와 바디를 OutputStream을 통해 클라이언트에 전송한다.
@mybloom
refactor: url에 맞는 파일 읽어 reponse body생성하도록 리팩토링, Path객체 생성 방법 변경 (#1)
9daef2c 
- Path객체 생성하는 방법 변경
- File.toPath()로 변경 : header에 추출한 url을 그대로 사용가능하여 변경.
- Path.of()는 /없도록 파싱하는 로직이 필요하여 변경.
@mybloom
build: lombok 의존성 추가 (#1)
1f1dfee
@mybloom
feat: 요구사항2. GET방식으로 회원가입 처리하기 위해 쿼리파라미터 파싱 후 User객체 생성 (#1)
e497ae7
@mybloom
feat: User 도메인에 setter 추가, 롬복 애너테이션 적용 (#1)
49ce56f
@mybloom
refactor: 쿼리파라미터 파싱로직 HttpRequestUtils.parseQueryString() Util클래스 메서드…
3803ba4 
… 이용하도록 변경 (#1)
@mybloom
feat: Request Header 클래스로 작성 (#1)
de046a0 
- RequestHeader 가 아래 3개의 속성 가지고 있다.
	RequestLine requestLine;
	List<String> requestHeaders;
	List<String> body;
- RequestLine은 httpMethod, url, httpVersion로 구성
@mybloom
refactor: RequestHandler 에서 사용자 요청에 대한 처리 단계 리팩토링 (#1)
aebaecf 
- request.handleUserRequest(in)에서 요청을 읽어 파싱하고
- response.makeResponse(requestLine.getUrl(), out)에서 응답 생성
@mybloom
feat: GET 방식 회원가입 후 Database.findAll()로 사용자 목록 log로 출력 (#1)
0d7268c 
- 요구사항에는 없었지만 확인차
@mybloom
feat: 요구사항3: POST방식으로 회원가입하기 (#1)
9e162e0
nathan29849 added a commit that referenced this pull request Jul 27, 2022
@nathan29849
feat: [#7] css, js, ico 등 html이 아닌 파일에 대해 GET 요청 처리
8ab7e8e
nathan29849 added a commit that referenced this pull request Jul 27, 2022
@nathan29849
Merge pull request #11 from nathan29849/feature/#7/css
e00698d 
feat: [#7] css, js, ico 등 html이 아닌 파일에 대해 GET 요청 처리
ku-kim
ku-kim approved these changes Jul 31, 2022
View reviewed changes
Copy link
Member

@ku-kim ku-kim left a comment

Choose a reason for hiding this comment

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

step5도 화이팅이에요 테리!

jinan159 added a commit that referenced this pull request Aug 7, 2022
@jinan159
Merge pull request #7 from jinan159/chapter34/step7
2e061bc 
3장 요구사항 7 - CSS 지원하기
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ku-kim ku-kim ku-kim approved these changes

Assignees
No one assigned
Labels
🕹 제출
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mybloom @ku-kim