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

[page types] Define page types for HTTP #22761

Closed
wbamberg opened this issue Dec 5, 2022 · 5 comments
Closed

[page types] Define page types for HTTP #22761

wbamberg opened this issue Dec 5, 2022 · 5 comments
Labels
Content:HTTP HTTP docs

Comments

@wbamberg
Copy link
Collaborator

wbamberg commented Dec 5, 2022

Part of #15539.

In https://docs.google.com/spreadsheets/d/14RX8EEKPpeYkP2cF5Y_ZKF3JpNJ_24OL7stvZWNJLIY/edit#gid=2083448819, in the "HTTP" tab, I've listed every page under https://developer.mozilla.org/en-US/docs/Web/HTTP and made an initial determination about the page type.

I've come up with the following types:

guide                                      38
HTTP content security policy directive     31
HTTP CORS error                            15
HTTP feature policy directive              29
HTTP header                               121
HTTP header attribute                       1
HTTP method                                 9
HTTP status code                           54
landing page                                4

I'm a bit unhappy that we have a single "HTTP header attribute" page, which is https://developer.mozilla.org/en-US/docs/web/http/headers/set-cookie/samesite. I wonder if we could fold it into https://developer.mozilla.org/en-US/docs/web/http/headers/set-cookie.

Otherwise this seems like a reasonable list.

@github-actions github-actions bot added Content:HTTP HTTP docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Dec 5, 2022
@wbamberg wbamberg mentioned this issue Dec 5, 2022
6 tasks
@teoli2003
Copy link
Contributor

I'm a bit unhappy that we have a single "HTTP header attribute" page, which is https://developer.mozilla.org/en-US/docs/web/http/headers/set-cookie/samesite. I wonder if we could fold it into https://developer.mozilla.org/en-US/docs/web/http/headers/set-cookie.

Yes, this page is difficult to navigate from and to.

I don't think we can fold it entirely into set-cookie, but I think we can deal with it by:

  1. Checking/Updating set-cookie to describe the modern value
  2. Merge the sections about dealing with problems into the guide about HTTP Cookies, eventually adding a second part specific to same-site cookies.

@sideshowbarker sideshowbarker removed the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Dec 6, 2022
@wbamberg
Copy link
Collaborator Author

wbamberg commented Dec 6, 2022

I don't think we can fold it entirely into set-cookie, but I think we can deal with it by:

  1. Checking/Updating set-cookie to describe the modern value

  2. Merge the sections about dealing with problems into the guide about HTTP Cookies, eventually adding a second part specific to same-site cookies.

I like this plan!

@teoli2003
Copy link
Contributor

teoli2003 commented Jan 16, 2023

I think we can move forward with this plan. (Great plan, @wbamberg!)

Just listing the entries here as checkboxes so we can track progress:

  • Update meta docs about these types
  • guide – 38 pages
  • http-csp-directive (HTTP content security policy directive) – 31 pages
  • http-cors-error (HTTP CORS error) – 15 pages
  • http-feature-policy-directive (HTTP feature policy directive) – 29 pages
  • http-header(HTTP header) – 121 pages
  • Special case for set-cookie
  • http-method(HTTP method) – 9 pages
  • http-status-code (HTTP status code) – 54 pages
  • landing page

@wbamberg
Copy link
Collaborator Author

@teoli2003 I have a dodgy script I can use to apply the page-type values we have agreed. The main thing blocking this is set-cookie - if you wanted to resolve this per #22761 (comment) that would be great.

@wbamberg
Copy link
Collaborator Author

With #26102 merged, we can close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTTP HTTP docs
Projects
None yet
Development

No branches or pull requests

3 participants