fetch icon indicating copy to clipboard operation
fetch copied to clipboard

Add http method to fetch timing info

Open sohomdatta1 opened this issue 1 year ago • 2 comments

This change will allow for the addition of the http method to the resource-timing spec

Explainer: https://github.com/sohomdatta1/rt-explainer Associated resource-timing PR: w3c/resource-timing#375

  • [ ] At least two implementers are interested (and none opposed):
  • [ ] Tests are written and can be reviewed and commented upon at:
  • [ ] Implementation bugs are filed:
    • Chromium: …
    • Gecko: …
    • WebKit: …
    • Deno (not for CORS changes): …
  • [ ] MDN issue is filed: …

(See WHATWG Working Mode: Changes for more details.)


Preview | Diff

sohomdatta1 avatar Apr 17 '23 12:04 sohomdatta1

The explainer link is a 404. Is it https://github.com/sohomdatta1/rt-explainer?

I think this should be called requestMethod or just method. (See also responseStatus.)

Also, a method can change due to redirects, but this won't pick that up.

annevk avatar Apr 17 '23 15:04 annevk

The explainer link is a 404. Is it https://github.com/sohomdatta1/rt-explainer?

That is the correct link, I have updated the description

I think this should be called requestMethod or just method. (See also responseStatus.)

Also, a method can change due to redirects, but this won't pick that up.

Can you point me to which cases the redirect would change the method being used? Also, does this mean we should restrict this behind CORS (given that reporting a different HTTP method could be considered a leak of whether or not a redirect occurred)?

sohomdatta1 avatar Apr 17 '23 16:04 sohomdatta1