Skip to content

Conversation

lforst
Copy link
Contributor

@lforst lforst commented Oct 25, 2023

No description provided.

github-actions bot and others added 4 commits October 24, 2023 10:46
[Gitflow] Merge master into develop
Profiling is currently unable to profile page loads which creates a gap
in the product as performance instrumentation seemingly can (via
synthetic transactions)

In order to bridge this gap, the plan is to provide a small JS snippet
that users can insert into their document which will initialize and
store a reference to the pageload profile on the sentry carrier. By
doing so we can hook back into our finishTransaction codepath and send a
profile associated to a transaction.

Would love to hear early thoughts on this approach from SDK maintainers
before I start adding test coverage.
Since there is an unknown elapsed time between document response and
profile start, setting profile timestamp = transaction start timestamp
can significantly skew our profile and cause alignment issue in the
product. This updates the pageload profiles to create independent
timestamps.
@lforst lforst requested review from Lms24 and mydea October 25, 2023 09:29
@lforst lforst force-pushed the prepare-release/7.75.1 branch from 0aa1d80 to 6fa0d7c Compare October 25, 2023 10:23
@lforst lforst merged commit 70598d6 into master Oct 25, 2023
@lforst lforst deleted the prepare-release/7.75.1 branch October 25, 2023 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants