whatwg / fetch

Fetch Standard
https://fetch.spec.whatwg.org/
Other
2.12k stars 333 forks source link

Add http method to fetch timing info #1632

Open sohomdatta1 opened 1 year ago

sohomdatta1 commented 1 year ago

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

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


Preview | Diff

annevk commented 1 year ago

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.

sohomdatta1 commented 1 year ago

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)?