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

[테리_쿠킴] 웹서버 2 단계 - GET으로 회원가입 기능 구현 #20

Merged
merged 9 commits into from Mar 24, 2022
Merged

[테리_쿠킴] 웹서버 2 단계 - GET으로 회원가입 기능 구현 #20

merged 9 commits into from Mar 24, 2022

Commits on Mar 23, 2022

  1. feat: 클라이언트 HTTP Request을 담은 Request 객체 생성 

    - 서버에서 InputStream in 을 통해 클라이언트 HTTP Request을 입력 받아 Request 객체를 생성한다.
- Request에는 method, uri, version, queryStringMap, RequestHandlerMap으로 파싱된다.
    @ku-kim
    ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    9fd1cd9 View commit details
    Browse the repository at this point in the history

  2. feat: GET으로 회원가입 기능 구현 

    - GET /user/create?queryString 의 예처럼 쿼리스트링으로 회원가입 포맷이 들어오면 해당 경로 uri 처리하며 User 생성
- queryString으로 들어온 값은 퍼센트 인코딩되어있기 때문에 HttpRequestUtils.parseQueryString에서 URLDecoder.decode 메서드를 사용해 디코드하여 한글 문제 해결
- HttpRequestUtilsTest.parseQueryString_korean 테스트 코드 추가
- User 생성 시 log.debug로 유저 생성 여부 로깅
    @ku-kim
    ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    55ca41e View commit details
    Browse the repository at this point in the history

  3. fix: HttpRequestUtils.parseQueryString() 매개변수 null일 때 NPE문제 발생과 해결 

    - query string이 null일 때 NPE발생하여 비어있는 Map 리턴하도록 수정
    @mybloom @ku-kim
    mybloom authored and ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    e71e17f View commit details
    Browse the repository at this point in the history

  4. fix: HttpRequestUtils.parseQueryString() 매개변수 null일 때 NPE문제 발생과 해결 

    - query string이 null일 때 NPE발생하여 비어있는 Map 리턴하도록 수정
    @mybloom @ku-kim
    mybloom authored and ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    e98f979 View commit details
    Browse the repository at this point in the history

  5. refactor: HttpRequestUtils.parseQueryString()메서드 간결화 

    - URLDecoder.decode()메서드 사용할 떄 "UTF-8" -> StandardCharsets.UTF_8 으로 변경하여 불필요한 예외처리 제거
    @ku-kim
    ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    6e4ee60 View commit details
    Browse the repository at this point in the history

  6. test: Request 클래스의 생성자 테스트 

    - 간단한 GET 요청일 때 Request 객체 생성 여부 확인
- 쿼리파라미터가 있는 GET 요청일 때 Request 객체 생성 여부 확인
    @ku-kim
    ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    91463de View commit details
    Browse the repository at this point in the history

  7. refactor: Request 클래스 가독성을 위한 메서드 추출, 구조 변겅 

    - Request(String requestLine, List<String> rawData) 생성자 구조 변경과 메서드 추출
    @ku-kim
    ku-kim committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    7d6d142 View commit details
    Browse the repository at this point in the history

  8. Merge remote-tracking branch 'origin/step2' into step2

    @mybloom
    mybloom committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    3b9d28d View commit details
    Browse the repository at this point in the history

  9. fix: 리팩토링하면서 처리하지 못한 HttpRequestUtils.parseQueryString() 

    매개변수 null일 때 NPE문제 발생과 해결

- query string이 null일 때 NPE발생하여 비어있는 Map 리턴하도록 수정
    @mybloom
    mybloom committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    6fe7fe0 View commit details
    Browse the repository at this point in the history