Closed turt2live closed 2 months ago
See https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods
We notably don't include the following:
CONNECT
TRACE
We add the following, though:
OPTIONS
PATCH
HEAD
We add them now instead of later to avoid relatively tiny PRs needing to go through heavyweight process. Instead, other layers can just use these verbs if and as needed.
It was briefly considered to use https://github.com/jrylan/http-method-enum instead, though the supply chain vulnerability risk doesn't feel worth it.
public
exported
See https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods
We notably don't include the following:
CONNECT- Matrix doesn't use this, and is unlikely to foreseeably use it.TRACE- Same as aboveWe add the following, though:
OPTIONS- Valid thing to use under https://spec.matrix.org/v1.10/client-server-api/#web-browser-clientsPATCH- Plausibly useful while we're here. May be used in the near-ish future.HEAD- Could be used by https://github.com/matrix-org/matrix-spec-proposals/pull/4120We add them now instead of later to avoid relatively tiny PRs needing to go through heavyweight process. Instead, other layers can just use these verbs if and as needed.
It was briefly considered to use https://github.com/jrylan/http-method-enum instead, though the supply chain vulnerability risk doesn't feel worth it.
Checklist
public/exportedsymbols have accurate TSDoc documentation.