-
Notifications
You must be signed in to change notification settings - Fork 55
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
Review request for HTTP Status Code in Resource Timing #757
Comments
I suggest using |
This is the kind of information that developers asked since quite a long time, if security and fetch people think that it is done in a way that doesn't enable security risks, then it looks like a valid addition, but iirc, previous attempts failed in the past. |
The explainer suggests that "The status code is behind CORS check", which would alleviate my concerns. That's the same check we use for the |
Yeah, as long as you don't go beyond the normal same-origin policy and its CORS extension in terms of information exposure it ought to be fine. (I vaguely recall prior attempts wanting to expose it all the time, which would be problematic.) |
Hi @abinpaul1 , we had a discussion in today's TAG meeting and are generally happy with this proposal. |
Wotcher TAG!
I'm requesting a TAG review of HTTP Status Code in Resource Timing.
Adds a field
responseStatusCode
to PerfomanceResourceTiming which holds an integer corresponding to HTTP status code returned when fetching the resource.Further details:
You should also know that...
We'd prefer the TAG provide feedback as :
💬 leave review feedback as a comment in this issue and @-notify @abinpaul1 @yoavweiss
The text was updated successfully, but these errors were encountered: