This repository has been archived by the owner on Mar 22, 2023. It is now read-only.
Translate Node.js v6.2.2 #379
Merged
Conversation
This file contains 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
| ### 주요 변경사항 | ||
| * **http**: | ||
| - `req.read(0)`는 특정 상황에서 들어오는 연결들이 멈추거나 타임아웃 될 수 있습니다. (Fedor Indutny) [#7211](https://github.com/nodejs/node/pull/7211) | ||
| - keep-alive 에이전트가 이미 끝났거나 끝난 모든 상황을 기다리기 위해 재사용할 소켓이 자유로워 질때, 그렇지 않으면 다음 요청은 이전의 연결 하나가 body에 전송이 완료 되기 전에 파서가 오류를 발생시키게 기록될 것입니다. (Fedor Indutny) [#7149](https://github.com/nodejs/node/pull/7149) |
There was a problem hiding this comment.
의미를 이해하기가 약간 어려운데요.
keep-alive 에이전트에서 소켓을 재사용하기 위해 소켓을 반환할 때 prefinish와 end 이벤트를 모두 기다립니다. 그렇지 않으면 이전 요청이 보디를 다 보내기 전에 다음 요청을 작성할 수 있으므로 파싱 오류가 발생할 수 있습니다.
는 어떨까요? https://github.com/nodejs/node/pull/7149를 보면 end와 prefinish가 이벤트의 이름으로 보여서 굳이 번역하지 않아도 될 것 같습니다.
|
다 봤습니다. |
|
리뷰내용 반영했습니다. 첫번째 리뷰는 조금 제가 다시 수정해봤습니다. 감사합니다. |
|
|
||
| ### 주요 변경사항 | ||
| * **http**: | ||
| - `req.read(0)`는 특정 상황에서 들어오는 연결들이 멈추거나 타임아웃 될 수 있습니다. (Fedor Indutny) [#7211](https://github.com/nodejs/node/pull/7211) |
There was a problem hiding this comment.
req.read(0)이 특정 상황에서 들어오는 연결을 멈추거나 타임아웃시킬 수 있었다는 의미로, 지금은 고쳤다는 이야기 같습니다.
There was a problem hiding this comment.
저도 그 부분이 좀 헷갈렸습니다. 리뷰 감사합니다.
-> 될 수 있어서 수정 되었습니다. 라고 추가할까요?
|
다 봤습니다. |
|
수정해서 커밋했습니다. |
|
|
||
| ### 주요 변경사항 | ||
| * **http**: | ||
| - `req.read(0)`는 특정 상황에서 들어오는 연결들이 멈추거나 타임아웃 될 수 있어서 수정되었습니다. (Fedor Indutny) [#7211](https://github.com/nodejs/node/pull/7211) |
There was a problem hiding this comment.
'req.read(0)는 특정 상황에서 들어오는 연결들이 멈추거나 타임아웃 될 수 있어서'라고 하면 주술 호응이 잘 안 되는 것 같습니다. 'req.read(0)이 특정 상황에서 들어오는 연결들을 멈추거나 타임아웃 시킬 수 있어서'는 어떻습니까?
There was a problem hiding this comment.
네 그렇게 수정하는게 읽기 좋겠네요.
수정해서 commit 했습니다.
감사합니다:)
Closed
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.