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

chore(http): Refresh headers docs i-r #36079

Merged

hamishwillee merged 10 commits into mdn:main from bsmth:http-headers-l

Oct 24, 2024

Member

bsmth commented Sep 27, 2024 •

edited

Loading

Description

This PR refreshes a few HTTP headers pages.

common changes:

Improve See also
Capitalize Yes/No in properties table
Normalize The HTTP **`Header-Name`** (request|response) header in first sentence
Add status text to HTTPStatus macros, e.g. {{HTTPStatus("406", "406 Not Acceptable")}}
Remove improper use of quotes in and around code
Avoid escaping angle brackets in directive names in favor of backticks:

- \<directive>
+ `<directive>`

Motivation

Keeping content up-to-date, fixing formatting, unifying writing conventions

Related pull requests:


          chore(http): Refresh headers docs i-r

c4361f0

bsmth requested a review from a team as a code owner

September 27, 2024 15:11

bsmth requested review from hamishwillee and removed request for a team

September 27, 2024 15:11

github-actions bot added Content:HTTP size/l labels

Contributor

github-actions bot commented Sep 27, 2024 •

edited

Loading

Preview URLs (20 pages)

External URLs (8)

URL: /en-US/docs/Web/HTTP/Headers/Link
Title: Link

https://www.iana.org/assignments/link-relations/link-relations.xhtml (1 time) (Note! This may be a new URL 👀)
https://www.rfc-editor.org/rfc/rfc7230.html (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/HTTP/Headers/Origin-Agent-Cluster
Title: Origin-Agent-Cluster

https://tc39.es/ecma262/ (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/HTTP/Headers/Refresh
Title: Refresh

https://lists.w3.org/Archives/Public/ietf-http-wg/2019JanMar/0197.html (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/HTTP/Headers/Proxy-Authorization
Title: Proxy-Authorization

https://www.iana.org/assignments/http-authschemes/http-authschemes.xhtml (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/HTTP/Headers/Referrer-Policy
Title: Referrer-Policy

https://en.wikipedia.org/wiki/HTTP_referer (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/HTTP/Headers/If-Unmodified-Since
Title: If-Unmodified-Since

https://en.wikipedia.org/wiki/Optimistic_concurrency_control (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/HTTP/Headers/Referer
Title: Referer

https://en.wikipedia.org/wiki/HTTP_referer (2 times) (Note! This may be a new URL 👀)

(comment last updated: 2024-10-23 10:44:34)

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/if-unmodified-since/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/if-unmodified-since/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/last-modified/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/location/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/location/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/if-unmodified-since/index.md Outdated Show resolved Hide resolved


          Apply suggestions from code review

0c13ac3

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/no-vary-search/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/observe-browsing-topics/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/origin/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/priority/index.md Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/priority/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/proxy-authorization/index.md Outdated Show resolved Hide resolved

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/proxy-authorization/index.md Outdated Show resolved Hide resolved

Collaborator

hamishwillee commented Sep 30, 2024

@bsmth Looks great. A few merged comments, and a few for discussion - all pretty minor

bsmth commented

View reviewed changes

files/en-us/web/http/headers/origin/index.md Outdated Show resolved Hide resolved

bsmth and others added 2 commits

October 1, 2024 11:57


          Apply suggestions from code review

7ceb86f

Co-authored-by: Hamish Willee <[email protected]>


          Update files/en-us/web/http/headers/origin/index.md

7cbf96f

github-actions bot added size/m and removed size/l labels

bsmth commented

View reviewed changes

files/en-us/web/http/headers/proxy-authorization/index.md Outdated Show resolved Hide resolved

bsmth added 2 commits

October 1, 2024 11:58


          Update files/en-us/web/http/headers/proxy-authorization/index.md

c0952a6


          Merge branch 'main' into http-headers-l

42993e0


          chore(http): Changes following reviewer feedback

2fa77c5

github-actions bot added size/l and removed size/m labels

hamishwillee reviewed

View reviewed changes

files/en-us/web/http/headers/if-unmodified-since/index.md Show resolved Hide resolved

github-actions bot added the merge conflicts 🚧 label

Contributor

github-actions bot commented Oct 14, 2024

This pull request has merge conflicts that must be resolved before it can be merged.

bsmth self-assigned this


          Merge branch 'main' into http-headers-l

dfe15b1

github-actions bot removed the merge conflicts 🚧 label

github-actions bot reviewed

View reviewed changes

files/en-us/web/http/headers/link/index.md Outdated Show resolved Hide resolved

bsmth and others added 2 commits

October 23, 2024 11:23


          Update files/en-us/web/http/headers/link/index.md

f63119e

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>


          chore(http): Add glossary links to request and response header terms

2828f1d

hamishwillee approved these changes

View reviewed changes

Collaborator

hamishwillee left a comment

Thanks very much @bsmth - LGTM now!

hamishwillee merged commit cadc98b into mdn:main

8 checks passed

bsmth deleted the http-headers-l branch

October 28, 2024 08:40

Member Author

bsmth commented Oct 28, 2024

Super, thank you, Hamish 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTTP size/l