Skip to content

ref: Add fast-path to fetchImpl and cleanup redundant iframe - #3341

Merged
kamilogorek merged 2 commits into
masterfrom
fetch-impl-fastpath
Mar 22, 2021
Merged

ref: Add fast-path to fetchImpl and cleanup redundant iframe#3341
kamilogorek merged 2 commits into
masterfrom
fetch-impl-fastpath

Conversation

@kamilogorek

Copy link
Copy Markdown
Contributor

@github-actions

github-actionsBot commented Mar 21, 2021

Copy link
Copy Markdown
Contributor

size-limit report

PathSize
@sentry/browser - CDN Bundle (gzipped)20.45 KB (+0.1% 🔺)
@sentry/browser - Webpack21.29 KB (+0.06% 🔺)
@sentry/react - Webpack21.33 KB (+0.05% 🔺)
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped)27.53 KB (+0.06% 🔺)

@kamilogorek
kamilogorek requested a review from a teamMarch 22, 2021 11:34
@kamilogorekkamilogorek changed the title ref: Add fast-path to fetchImplref: Add fast-path to fetchImpl and cleanup redundant headerMar 22, 2021
@kamilogorekkamilogorek changed the title ref: Add fast-path to fetchImpl and cleanup redundant headerref: Add fast-path to fetchImpl and cleanup redundant iframeMar 22, 2021

@lobsterkatielobsterkatie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kamilogorek
kamilogorek merged commit 2d6196b into masterMar 22, 2021
@kamilogorek
kamilogorek deleted the fetch-impl-fastpath branch March 22, 2021 13:43
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.

3 participants

@kamilogorek@lobsterkatie@iker-barriocanal