getsentry / getsentry/sentry-javascript

Keep min. 30 seconds for replay in error mode

Ouverte
#6,908 15 commentaires 0 réactions 1 personne assignée Réclamée par @billyvg Voir sur GitHub
Browser Improvement Replays
Langage dominant
TypeScript
Étoiles
8.7k
Forks
1.8k
Merge moyen
1 j 17 h
PR mergées (30 j)
515

Description

Currently, when in error mode, we keep _up to_ 60 seconds of replay recording for when an error happens.

Basically, what happens is:

* rrweb is instructed to do a full checkout every 60 seconds
* The event buffer will generally discard anything buffered before when it receives a full checkout
* When an error occurs, we send the whole buffer
* This means we send anything from 0-60 seconds of replay data

IMHO this can be unexpected (it was unexpected to me, I thought it would always keep the last 60s 😅 ). I propose that we change this to keep a min. of 60 seconds (except at the very beginning of the replay, of course. So we'd send anywhere from 60-120 seconds of replay data.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.