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 Replay Incorrectly Shows Page DOM Content Duplicates #45101

Closed
kpc-seth opened this issue Feb 24, 2023 · 11 comments
Closed

Session Replay Incorrectly Shows Page DOM Content Duplicates #45101

kpc-seth opened this issue Feb 24, 2023 · 11 comments
Assignees

Comments

@kpc-seth
Copy link

Environment

SaaS (https://sentry.io/)

Version

No response

Link

https://kpc-cf.sentry.io/replays/javascript:b4bb3c63380e429a96bcfa8b8ba54143/?project=4504575536201728&query=&referrer=%2Freplays%2F&statsPeriod=14d&yAxis=count%28%29

DSN

No response

Steps to Reproduce

  1. Go to replay
  2. Play replay
  3. Look at window and see data duplicated in scroll (see attached image under Actual Result)

Expected Result

Window should not show duplicates.

Actual Result

SentryDuplicateData

@getsantry
Copy link
Contributor

getsantry bot commented Feb 24, 2023

Assigning to @getsentry/support for routing, due by Monday, February 27th at 1:52 pm (sfo). ⏲️

@getsantry
Copy link
Contributor

getsantry bot commented Feb 25, 2023

Routing to @getsentry/replay for triage, due by Tuesday, February 28th at 11:00 am (sfo). ⏲️

@billyvg
Copy link
Member

billyvg commented Feb 27, 2023

@kpc-seth Can you trying updating to the latest SDK version (7.39.0)?

@kpc-seth
Copy link
Author

kpc-seth commented Mar 2, 2023

Done. Issue persists.

@kpc-seth
Copy link
Author

kpc-seth commented Mar 3, 2023

Issue persists with version 7.40 as well

@kpc-seth
Copy link
Author

Are there any updates related to root cause and/or fix? Thanks.

@billyvg
Copy link
Member

billyvg commented Mar 16, 2023

@kpc-seth Sorry this has taken awhile, I'm still looking into this - I see that there are some large mutations happening (> 10k) but haven't been able to confirm yet if this is the root cause. Would you be able to provide me with a test account so I can test some potential fixes? My email is billy at sentry.io if so

@billyvg
Copy link
Member

billyvg commented Mar 16, 2023

getsentry/rrweb#82 should fix the duplicates on the player side, but I would still like to investigate why it is recording duplicate DOM elements.

@kpc-seth
Copy link
Author

@billyvg - which test accounts do you need? Access to our Sentry account? Access to our application's test environment to run a use case that reproduces the issue in Sentry? Both? Thanks.

@billyvg
Copy link
Member

billyvg commented Mar 20, 2023

@kpc-seth Just the latter would be great (access to your application -- it doesn't need to have Sentry or Replay enabled)

@billyvg
Copy link
Member

billyvg commented Mar 20, 2023

@kpc-seth The replay should now play correctly, but if you could email me credentials for a test account, I would still be interested in investigating this issue from the recording side. Thanks.

@billyvg billyvg closed this as completed Mar 20, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants