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
feat(replay): Add responseStatus
, decodedBodySize
to perf entries
#7613
Conversation
Add a few more properties to `resource` performance entries (e.g. script, img, etc). * Adds `responseStatus` which is new in Chrome 109 for status code * Adds `decodedBodySize` so that we can tell user if their resource was compressed or not.
size-limit report 📦
|
responseStatus?: number; | ||
}; | ||
|
||
export type PaintData = undefined; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems a bit weird? Should this be undefined
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it is a bit weird but we don't attach data for paint, so those breadcrumbs will not have a data
attribute at all
Add a few more properties to
resource
performance entries (e.g. script, img, etc).responseStatus
which is new in Chrome 109 for status codedecodedBodySize
so that we can tell user if their resource was compressed or not.