HTTP 웹서버 미션 1단계 제출합니다. #51
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Create new User entity by inserting parameter map into the constructor
feat: POST request to process with creating new user
feat: add junction logic for HttpMethod and support 302 redirect
|
수고하셨어요. 끝까지 즐겁게 열심히 해주셔서 엄청 좋네요. |
honux77
reviewed
Jul 1, 2020
| @@ -9,3 +9,15 @@ | |||
| ## 온라인 코드 리뷰 과정 | |||
| * [텍스트와 이미지로 살펴보는 코드스쿼드의 온라인 코드 리뷰 과정](https://github.com/code-squad/codesquad-docs/blob/master/codereview/README.md) | |||
| * [동영상으로 살펴보는 코드스쿼드의 온라인 코드 리뷰 과정](https://youtu.be/a5c9ku-_fok) | |||
|
|
|||
| ## 학습한 점 | |||
| * BufferedReader는 InputStream 등 다양한 입력 스트림을 편하게 관리해주는 클래스이다. | |||
There was a problem hiding this comment.
편하게도 맞는데 성능도 있습니다. 말 그대로 버퍼를 사용하기 때문에 성능이 좋아지고, 추가로 다른 처리를 쉽게 할 수 있는 거죠. OS의 입출력 버퍼에 대해서도 공부해 보세요.
There was a problem hiding this comment.
@honux77 운영체제도 공부할 꺼리가 되는 것 같네요. 버퍼에 대해 공부해보도록 하겠습니다.
honux77
reviewed
Jul 1, 2020
| ## 학습한 점 | ||
| * BufferedReader는 InputStream 등 다양한 입력 스트림을 편하게 관리해주는 클래스이다. | ||
| * HTTP Request의 마지막 라인은 ```""``` 로 빈 문자열이 들어간다. | ||
| * index.html로 연결을 시도했는데 favicon.ico에 대한 연결도 브라우저가 자동으로 시도한다. 이는 아이콘을 가져오기 위한 것으로 보인다. |
|
@honux77 감사합니다 호눅스. 웹서버 2단계 미션도 이번 주 주말 중으로 끝내서 PR 드리겠습니다. |
|
@honux77 자바 프로그래밍 Next Step 책을 시작해보려고 합니다. |
|
LGTM 👍 |
|
오 이런 훈훈한 피알이 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
같이 고민하고자 하는 사항
loggedIn이true로 쿠키값이 발송되지만, 로그인한 상태에서/user/list에 접근하면loggedIn이false인 쿠키값이 발송되어 사용자 리스트를 반환하지 못하고 있습니다. 해결책이 마땅하지가 않아서 함께 고민해주실 것을 부탁드립니다.