Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

fix: Order of capturing replay events and uploading replay is a bit wrong#110

Merged
billyvg merged 1 commit intomainfrom
fix/move-capture-replay-before-sending-replay
Jul 18, 2022
Merged

fix: Order of capturing replay events and uploading replay is a bit wrong#110
billyvg merged 1 commit intomainfrom
fix/move-capture-replay-before-sending-replay

Conversation

@billyvg
Copy link
Copy Markdown
Member

@billyvg billyvg commented Jul 18, 2022

  • captureReplay needs to happen before sendReplay
  • captureEvent should only happen after a successful sendReplay
  • fix sendReplay promises where it now awaits the retries before resolving

…rong

* `captureReplay` needs to happen before `sendReplay`
* `captureEvent` should only happen after a *successful* `sendReplay`
* fix `sendReplay` promises where it now awaits the retries before resolving
@billyvg billyvg force-pushed the fix/move-capture-replay-before-sending-replay branch from ac10703 to 87d09fa Compare July 18, 2022 21:54
@billyvg billyvg merged commit f39763b into main Jul 18, 2022
@billyvg billyvg deleted the fix/move-capture-replay-before-sending-replay branch July 18, 2022 22:49
mydea pushed a commit to getsentry/sentry-javascript that referenced this pull request Nov 23, 2022
…rong (getsentry/sentry-replay#110)

* `captureReplay` needs to happen before `sendReplay`
* `captureEvent` should only happen after a *successful* `sendReplay`
* fix `sendReplay` promises where it now awaits the retries before resolving
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant