404-DriverNotFound / 200-OK

jikang, yunslee, ykoh project for Pass this project
2 stars 1 forks source link

Feat: HttpMessage 프로토타입 구현 #17

Closed ykoh42 closed 3 years ago

ykoh42 commented 3 years ago

WHAT?

http request 파싱과 http response 전송 프로토타입을 구현함. 아주 최소한 내용만 구현한 것이고 동작만 가능하게 구현하였음.

WHY?

http message의 일련의 과정들을 이해하기 위해.

TESTING

  1. Make
  2. 브라우저에서 http://localhost

ISSUE and REFERENCE

Resolves: #16 See also: #7

@PennyBlack2008 추가한 부분이 겹치는 부분이 있어서 확인해주어야 함

PennyBlack2008 commented 3 years ago
  1. Compile 확인.
  2. 예전에 테스트용으로 만들었던 코드 지운 것이 문제없음 확인.
  3. HTTP request 부분이 이해를 못했어서 코드리뷰 해줬으면 좋겠음.
exgs commented 3 years ago

PR (늦은) 코멘트

  1. HttpMessageResponse(const HttpMessageRequest& request) 생성자 코드를 보면,

    HttpMessageResponse::HttpMessageResponse(const HttpMessageRequest& request)
    : mRequest(request)
    {
    // if (mReasonPhrase.empty())
    // {
    //  mReasonPhrase[100] = "Continue";
    //  mReasonPhrase[101] = "Switching Protocols";
     // ...
     // }
    }
    1. 이런 부분이 있는데, 위 조건문이 동작하려면, if문 위에 mReasonPhrase(std::map<int, string>) 이 설정되는 것이 예쩡되어있는건가???
    2. mReasonPhrase는 Response Header로 전달할 key-value의 집합이야??
  2. [Type: Question] vscode extension Anchor를 사용하고 있는데, main.cpp 에서 STUB 라는 이름으로 Anchor 설정을 해놨는데 어떤 의미인거지? -> 구글링 했을 때, "Used for generated default snippet" 이런 느낌으로 설명되어있고, snippet은 재사용 가능한 코드 묶음. 정도로 해석되는데 잘 모르겠음

  3. +추가사항

    이랬으면, 좀 더 좋았을 꺼 같네

    이메일이 한번 뒤바뀌어서, gitkraken에서는 다소 이해하기 어렵게 git log graph가 설정됨.
    commit: Fix: 버퍼사이즈 수정 에서 Pull Request Merge 되면 좀더 깔끔하게 됬을꺼같은데, 그 부분이 조금 아쉬움

    Screen Shot 2021-04-27 at 11 25 15 AM

    깃허브 insight인데, 여기서도 PR이 됬다는것을 그래프로 알아보기 힘들게 표현된 듯.


다른 것들은 잘 동작하는 것으로 보임.

+ps: issue, PR 놓치지않게 @ 태그 달게 @kohyounghwan, @PennyBlack2008

ykoh42 commented 3 years ago

@exgs

  1. HttpMessageResponse 클래스에서 사용할 map 컨테이너인데 static으로 만들어서 딱 한번만 정의됐으면 좋겠는데 방법이 잘 안떠오르네.. + 코드 작성하면서 계속 컴파일이 안되어서 일단 주석처리함
  2. 스켈레톤코드 느낌으로 STUB라고 작성해두었음 임시코드 같은느낌..?
  3. 요거 나도 궁금했었는데머지되면 그래프가 붙어야되는것같은데 안붙어서 왜 안붙지 하고 있었는데 방법을 찾아봐야할듯..
exgs commented 3 years ago

모두 확인! @kohyounghwan