Skip to content

Conversation

@lforst
Copy link
Contributor

@lforst lforst commented Nov 30, 2023

No description provided.

github-actions bot and others added 21 commits November 28, 2023 11:09
[Gitflow] Merge master into develop
Resource size can be null in some browsers, so don't set it if it's null
(only set integers).
Add a guard to avoid adding the Sentry Vite plugin for source
maps upload in dev mode.

Reason: The vite plugin ships with a Sentry SDK. This SDK conflicts with
the Astro server-side SDK. For example for some reason, it skews up the
line numbers of stack frames. Not exactly sure why that's happening but
my best guess is that the plugin SDK somehow changes code (maybe we even
inject debugIds or release values or something like this) that mixes up
line numbers.

Anyway, I think it's generally more correct to not use the plugin in dev
mode.
vendored in from
https://github.com/GoogleChrome/web-vitals/tree/7f0ed0bfb03c356e348a558a3eda111b498a2a11

Note that this is on web vitals `v3.0.4`. We need to update to latest
web vitals version - will do this in a follow up PR.
Not adding any tests for `createMetricEnvelope` yet because the type for
`metricAggregate` is going to change, just added it as boilerplate.

In the next PR I'll be implementing a metrics aggregator class which
lives on the client. This is responsible for bucketing metrics and
incrementally flushing them out.

https://develop.sentry.dev/delightful-developer-metrics/
https://develop.sentry.dev/delightful-developer-metrics/sending-metrics-sdk/
@lforst lforst requested a review from Lms24 November 30, 2023 10:02
@lforst lforst merged commit 22960f5 into master Nov 30, 2023
@lforst lforst deleted the prepare-release/7.84.0 branch November 30, 2023 10:39
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.

6 participants