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.

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

[ko] HTTP Status 204, 404 수정, vary 헤더 수정 외 #11669

Merged
merged 1 commit into from
Mar 5, 2023

Conversation

wisedog
Copy link
Contributor

@wisedog wisedog commented Feb 16, 2023

  • HTTP Header vary 헤더 최신화
  • HTTP Header Last-Modified 헤더 최신화
  • HTTP Header Content-Length 헤더 최신화
  • HTTP Status Code 204 최신화
  • HTTP Status Code 404 최신화

@wisedog wisedog requested a review from a team as a code owner February 16, 2023 13:43
@wisedog wisedog requested review from yechoi42 and removed request for a team February 16, 2023 13:43
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Feb 16, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 16, 2023

Preview URLs

Flaws (4)

Note! 3 documents with no flaws that don't need to be listed. 🎉

URL: /ko/docs/Web/HTTP/Headers/Content-Length
Title: Content-Length
Flaw count: 2

  • macros:
    • /ko/docs/Glossary/Payload_header does not exist but fell back to /en-US/docs/Glossary/Payload_header
    • /ko/docs/Glossary/CORS-safelisted_response_header does not exist but fell back to /en-US/docs/Glossary/CORS-safelisted_response_header

URL: /ko/docs/Web/HTTP/Headers/Last-Modified
Title: Last-Modified
Flaw count: 2

  • macros:
    • /ko/docs/Glossary/Representation_header does not exist but fell back to /en-US/docs/Glossary/Representation_header
    • /ko/docs/Glossary/CORS-safelisted_response_header does not exist but fell back to /en-US/docs/Glossary/CORS-safelisted_response_header
External URLs (5)

URL: /ko/docs/Web/HTTP/Headers/Vary
Title: Vary


URL: /ko/docs/Web/HTTP/Status/404
Title: 404 Not Found


URL: /ko/docs/Web/HTTP/Status/204
Title: 204 No Content

(comment last updated: 2023-02-20 22:45:16)

Copy link
Member

@yechoi42 yechoi42 left a comment

Choose a reason for hiding this comment

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

안녕하세요 @wisedog 님! 리뷰 코멘트 남겨두었습니다 :) 확인해보시고 저를 다시 멘션하거나, 제게 다시 리뷰 요청 주세요~

Comment on lines 13 to 14
`Vary` 헤더는 {{HTTPStatus("200")}} `OK` 응답과 동일하게 {{HTTPStatus("304")}}
`Not Modified` 응답에서도 설정되어야 합니다.
Copy link
Member

Choose a reason for hiding this comment

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


## 호환성 참고사항

- `204 No Content`는 본문 없는 응답을 위한 상태 코드이지만, 서버에서 잘못되게 본문을 포함한 응답을 전달하는 경우가 존재할 수 있습니다. HTTP는 이런 경우를 사용자 에이전트 자의적으로, 서로 다르게 처리하는 것을 허용하고 있습니다. [이에 대한 토론은 여기서 확인할 수 있습니다.](https://github.com/httpwg/http-core/issues/26) 보통 지속 연결에서 볼 수 있는 문제로, 잘못 포함된 본문이 이후 요청에 대한 별도의 응답을 담고 있을 수 있습니다.
- 이 상태코드는 본문 없는 응답을 위한 상태 코드이지만, 서버에서 잘못되게 본문을 포함한 응답을 전달하는 경우가 존재할 수 있습니다. 이 프로토콜은 사용자 에이전트 다양한 방법으로 처리하는 것을 허용하고 있습니다. [이에 대한 토론은 여기서 확인할 수 있습니다.](https://github.com/httpwg/http-core/issues/26) 보통 지속 연결에서 볼 수 있는 문제로, 잘못 포함된 본문이 이후 요청에 대한 별도의 응답을 담고 있을 수 있습니다.

Apple Safari는 잘못 포함된 모든 데이터를 거부합니다. Google Chrome과 Microsoft Edge는 잘못된 데이터의 최대 4바이트를 검사한 후, 유효한 별도의 요청을 찾지 못한 경우 폐기합니다. Firefox는 최대 1킬로바이트를 검사합니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Apple Safari는 잘못 포함된 모든 데이터를 거부합니다. Google Chrome과 Microsoft Edge는 잘못된 데이터의 최대 4바이트를 검사한 후, 유효한 별도의 요청을 찾지 못한 경우 폐기합니다. Firefox는 최대 1킬로바이트를 검사합니다.
Apple Safari는 잘못 포함된 모든 데이터를 거부합니다. Google Chrome과 Microsoft Edge는 잘못된 데이터의 최대 4바이트를 검사한 후, 유효한 별도의 요청을 찾지 못한 경우 폐기합니다. Firefox는 1킬로바이트 이상을 검사합니다.

기여해주신 부분은 아니지만 원문과 비교했을 때 잘못 해석된 부분을 발견해 수정 코멘트 남깁니다. in exceed of 의 의미를 바로잡았습니다.

---

{{HTTPSidebar}}

HTTP **`404 Not Found`** 클라이언트 오류 응답 코드는 서버가 요청받은 리소스를 찾을 수 없다는 것을 의미합니다. 404 페이지를 띄우는 링크는 대체로 브로큰 링크(broken link) 또는 데드 링크(dead link)라고 부르며, [link rot](https://en.wikipedia.org/wiki/Link_rot) 대상일 수도 있습니다.

404 상태 코드는 리소스가 일시적, 또는 영구적으로 사라졌다는 것을 의미하지는 않습니다. 리소스가 영구적히 삭제되었다면 404 상태 코드 대신 {{HTTPStatus(410)}} (Gone) 상태 코드가 쓰여야 합니다.
404 상태 코드는 리소스가 일시적, 또는 영구적으로 사라졌다는 것을 의미하지는 않습니다. 리소스가 영구적으로 삭제되었다면 404 상태 코드 대신 {{HTTPStatus(410)}} (Gone) 상태 코드가 쓰여야 합니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
404 상태 코드는 리소스가 일시적, 또는 영구적으로 사라졌다는 것을 의미하지는 않습니다. 리소스가 영구적으로 삭제되었다면 404 상태 코드 대신 {{HTTPStatus(410)}} (Gone) 상태 코드가 쓰여야 합니다.
404 상태 코드는 리소스가 일시적이든 영구적이든 '사라졌다'는 것을 의미합니다. 리소스가 영구적으로 '삭제'되었다면 404 상태 코드 대신 {{HTTPStatus(410)}} (Gone) 상태 코드가 쓰여야 합니다.

not whether(~이든 아니든)의 의미를 살려 수정 제안 드려봅니다. 우리말에서는 404와 410의 차이인 '사라짐'과 '삭제'를 알기 어려운 듯해 홑따옴표로 강조해보았습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아하! 감사합니다!

Content-Length: <length>
```

## 디렉티브
## 지시자
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
## 지시자
## 지시어

https://github.com/mdn/translated-content/blob/main/docs/ko/guides/glossary-guide.md 에 따라 지시어로 수정합니다.

</tr>
<tr>
<th scope="row">{{Glossary("Forbidden header name")}}</th>
<th scope="row">{{Glossary("Forbidden header name", "금지된 헤더 이름")}}</th>
Copy link
Member

Choose a reason for hiding this comment

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

32번째 줄 메크로도 수정부탁드려요!
{{Glossary("CORS-safelisted response header")}}

Copy link
Member

Choose a reason for hiding this comment

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

  • 45번째줄도 지시자 ->지시어로 수정해주심 될 것 같습니다 :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

지시자군요. 이 부분은 기존 번역이라 딱히 신경을 쓰지 않았는데 앞으로는 좀 더 잘 챙겨봐야겠네요. 감사합니다.

- HTTP Header `vary` 헤더 최신화
- HTTP Header `Last-Modified` 헤더 최신화
- HTTP Header `Content-Length` 헤더 최신화
- HTTP Status Code `204` 최신화
- HTTP Status Code `404` 최신화
@wisedog
Copy link
Contributor Author

wisedog commented Feb 20, 2023

안녕하세요 @wisedog 님! 리뷰 코멘트 남겨두었습니다 :) 확인해보시고 저를 다시 멘션하거나, 제게 다시 리뷰 요청 주세요~

@yechoi42 좋은 리뷰 다시 한 번 감사합니다!

@yechoi42 yechoi42 merged commit 18996c0 into mdn:main Mar 5, 2023
@wisedog wisedog deleted the http/headers-1 branch March 5, 2023 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants