Skip to content

fix(release health): Only create sessions if the correct methods are defined - #3281

Merged
lobsterkatie merged 1 commit into
masterfrom
kmclb-try-catch-start-session
Mar 5, 2021
Merged

fix(release health): Only create sessions if the correct methods are defined#3281
lobsterkatie merged 1 commit into
masterfrom
kmclb-try-catch-start-session

Conversation

@lobsterkatie

Copy link
Copy Markdown
Member

There are edge cases (see #3207 and #3234) where the version of @sentry/browser loaded on a page can be >= 6.0.0 and the version of @sentry/hub loaded can be < 5.27.0. When this happens, the former calls hub.startSession() but the latter doesn't have such a method, and an error is thrown. This protects against that possibility.

@github-actions

github-actionsBot commented Feb 25, 2021

Copy link
Copy Markdown
Contributor

size-limit report

PathSize
@sentry/browser - CDN Bundle (gzipped)20.29 KB (+0.05% 🔺)
@sentry/browser - Webpack21.19 KB (+0.06% 🔺)
@sentry/react - Webpack21.23 KB (+0.06% 🔺)
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped)27.37 KB (+0.03% 🔺)

@lobsterkatie
lobsterkatieforce-pushed the kmclb-try-catch-start-session branch from 805b3c5 to 48b53f5CompareMarch 1, 2021 17:39
@lobsterkatie
lobsterkatieforce-pushed the kmclb-try-catch-start-session branch from 48b53f5 to 99440a3CompareMarch 3, 2021 23:32
@lobsterkatie
lobsterkatie merged commit 5997e04 into masterMar 5, 2021
@lobsterkatie
lobsterkatie deleted the kmclb-try-catch-start-session branch March 5, 2021 03:19
This was referenced Mar 11, 2021
This was referenced Mar 15, 2021
Sign up for freeto 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.

2 participants

@lobsterkatie@kamilogorek