fix(browser): Minimum time for initial flush breaks replays for non-SPA #9301

[chetan-187]

fix(browser): Minimum time for initial flush breaks replays for non-SPA #9301

This implementation is completely dependent on browser functionality. At least this will save some replay events which were getting lost.

[mydea]

Sadly we cannot do this, because adding such a handler disables back-forward cache, which we don't want to do for all users by default.

We are currently looking into improving the replay experience for MPAs generally, which will include some way to set replay up easily that works well for MPAs.

You can add this handler yourself, or you could try to initialize replay like this:

replayIntegration({
  minReplayDuration: 0,
  flushMinDelay: 200,
  flushMaxDelay: 250,
})

Which should also work much better for a MPA. I've experimented a bit with this and this seemed to work reasonably well for me, even without the unload handler!

[chetan-187]

No problem @mydea. Handling MPAs is always a tough one.
I have been looking into the sentry's code for the past weeks, there is always a learning curve to understand a new code base 🙌.

[mydea]

You can follow progress on this issue here: #9490 - it is on our radar to improve this! I'll close this in the meanwhile, we'll work on providing a better way to configure this in the near future!