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

Add HTTP QUERY method #33226

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add HTTP QUERY method #33226

wants to merge 1 commit into from

Conversation

chicoxyzzy
Copy link

@chicoxyzzy chicoxyzzy commented Apr 23, 2024

Description

This PR adds an HTTP QUERY method, which is still an IETF Internet-Draft (https://www.ietf.org/archive/id/draft-ietf-httpbis-safe-method-w-body-02.html)

Motivation

HTTP QUERY method is being added to different implementations like Node.js

Additional details

Related issues and pull requests

mdn/browser-compat-data#22920

@github-actions github-actions bot added Content:HTTP HTTP docs size/s 6-50 LoC changed labels Apr 23, 2024
@chicoxyzzy
Copy link
Author

chicoxyzzy commented Apr 23, 2024

I marked this as a draft, as AFAIU it also needs a browser-compat data (PR) and a few corrections.

I need help with these:

  • How can I mark the method as a draft on the methods page and on the QUERY method page?
  • I see that all the methods use {{speceifications}}, but I don't understand how they point to the correct spec. How can I use a draft to point to the proper URL?
  • Will {{HTTPMethod("QUERY")}} on the GET and POST doc pages just magically work or I should edit more files to make it work?

@github-actions github-actions bot added size/m 51-500 LoC changed and removed size/s 6-50 LoC changed labels Apr 23, 2024
title: QUERY
slug: Web/HTTP/Methods/QUERY
page-type: http-method
# browser-compat: http.methods.QUERY
Copy link
Author

Choose a reason for hiding this comment

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

uncomment when mdn/browser-compat-data#22920 is merged

title: QUERY
slug: Web/HTTP/Methods/QUERY
page-type: http-method
# browser-compat: http.methods.QUERY
Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change
# browser-compat: http.methods.QUERY

@pepelsbey pepelsbey added the waiting for implementations Waiting for feature to be implemented in browsers label Apr 23, 2024
@pepelsbey
Copy link
Contributor

Given that you’re…

Still exploring if the method is available in browsers

The MDN is not the right place for this method for now. Let’s wait until the first implementation.

@chicoxyzzy
Copy link
Author

Let’s wait until the first implementation.

Makes sense. Should I close this one or just leave it open and update when (or if) it will be shipped in browsers?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTTP HTTP docs size/m 51-500 LoC changed waiting for implementations Waiting for feature to be implemented in browsers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants