Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Session stops instead of pausing when inactive for 5 minutes and doesn't resume on activity #7929

Closed
3 tasks done
Kobby-Bawuah opened this issue Apr 21, 2023 · 8 comments
Closed
3 tasks done
Assignees
Labels
Package: replay Issues related to the Sentry Replay SDK Type: Bug

Comments

@Kobby-Bawuah
Copy link

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/browser

SDK Version

7.48

Framework Version

No response

Link to Sentry event

No response

SDK Setup

tracesSampleRate: 1.0,
replaysSessionSampleRate: 0,
replaysOnErrorSampleRate: 1.0,

Steps to Reproduce

Set up replay
Run you app
After 5 minutes of inactivity replay stops and does not restart even if more errors occur.

Expected Result

The session should not be stopped when it becomes inactive but paused. When activity resumes, the replay should also resume.

This was confirmed here: #7192 (comment)

Actual Result

The replay does not start even if an event is thrown or the page is refreshed (hitting F5 or CTRL + F5). The only way to make replays work again is to open another tab

Screenshot 2023-04-19 at 6 20 59 PM

@mydea
Copy link
Member

mydea commented Apr 21, 2023

Thanks for opening this! I could reproduce this - I think we'll try to fix this in the way of https://github.com/getsentry/sentry-javascript/pull/7768/files#r1173511991, where we are already touching this a bit.

@brotherko
Copy link

Hi @mydea what is the progress? This makes my app basically unable to generate replay

@mydea
Copy link
Member

mydea commented Apr 24, 2023

Hi @mydea what is the progress? This makes my app basically unable to generate replay

Hi @brotherko, this is currently in progress, I'll update you when this has been released/fixed!

@mydea mydea added the Package: replay Issues related to the Sentry Replay SDK label Apr 24, 2023
@mydea
Copy link
Member

mydea commented Apr 27, 2023

Hey @brotherko ,

this should be fixed in v7.50.0!

@mydea mydea closed this as completed Apr 27, 2023
@egissan
Copy link

egissan commented May 19, 2023

Hi @mydea

This issue came from our report and we can still observe similar behavior with v7.52.1 (@sentry/browser)

Steps to reproduce:

  1. Invoke action which triggers error -> error appears in Sentry with recording
  2. Minimize tab
  3. Wait for ~30mins
  4. Open minimized tab
  5. Trigger same action again -> error appears in Sentry, but without recording

We have "debug: true" flag set and this error shows up in console
image

Expected behavior:
Error replays should be recorded for the entire lifetime of the application. In our case browser tab can be alive for days or even weeks.

Thanks

@billyvg billyvg reopened this May 25, 2023
@egissan
Copy link

egissan commented May 30, 2023

@mydea @billyvg any updates on this?

@billyvg
Copy link
Member

billyvg commented May 30, 2023

@egissan I can confirm this is an issue, I will write up my findings in a new issue. Essentially once you capture a replay w/ onError sampling, it will stop after the replay ends until the user refreshes. I will talk with the team to see how we want to proceed.

@billyvg
Copy link
Member

billyvg commented May 30, 2023

@egissan I'm going to close this issue in favor of #8257 - please follow along there, thanks!

@billyvg billyvg closed this as completed May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: replay Issues related to the Sentry Replay SDK Type: Bug
Projects
None yet
Development

No branches or pull requests

5 participants