Skip to content

Conversation

waltjones
Copy link
Contributor

Description of the change

Helps the application correctly align playback by adding a noop event to the end of the recording.

Type of change

  • Bug fix (non-breaking change that fixes an issue)

'rrweb-replay-events',
{
eventType: 5,
json: JSON.stringify({tag: "replay.end", payload: {}}),
Copy link

@shankj3 shankj3 Jul 11, 2025

Choose a reason for hiding this comment

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

This won't help us in the case of someone closing a tab, right?
Ie previous recording events have already been sent for this replay, then one more is going to be sent in the near future but tab is closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This span either gets sent in whole, or not at all. The purpose of adding this event is so that the player extends its playback until the triggering occurrence occurs. (It's almost like giving the occurrence an rrweb event.)

@waltjones waltjones merged commit 0cfead5 into master Jul 11, 2025
4 checks passed
@waltjones waltjones deleted the waltjones/rrweb-end-event branch July 11, 2025 20:14
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.

2 participants