Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

avoid accessing performance APIs if fetch is untracked #71164

Merged

Conversation

ztanner
Copy link
Member

@ztanner ztanner commented Oct 11, 2024

If a request is marked as internal, we don't record fetch metrics, so we don't need to track a fetchStart. We're investigating cases where performance APIs might be unavailable in particular runtimes and can safely exclude internal requests from being subject to the possibility of it not being available.

@ijjk ijjk added created-by: Next.js team PRs by the Next.js team. type: next labels Oct 11, 2024
Copy link
Member Author

ztanner commented Oct 11, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ztanner and the rest of your teammates on Graphite Graphite

@ztanner ztanner force-pushed the 10-11-avoid_accessing_performance_apis_if_fetch_is_untracked branch from 55e4f2f to 46eef29 Compare October 11, 2024 21:48
@ztanner ztanner marked this pull request as ready for review October 11, 2024 21:50
@ztanner ztanner requested a review from gnoff October 11, 2024 21:50
@ijjk
Copy link
Member

ijjk commented Oct 11, 2024

Tests Passed

@ztanner ztanner merged commit 46025af into canary Oct 11, 2024
104 of 110 checks passed
@ztanner ztanner deleted the 10-11-avoid_accessing_performance_apis_if_fetch_is_untracked branch October 11, 2024 22:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 26, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants