-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
User Session Recording Results in Error #21447
Comments
Thank you very much for the reproduction steps @dehli. It looks like the latest version of our web capture tool (rrweb) made the changes causing this issue in a PR that was merged in the latest release: rrweb-io/rrweb#1351. We upgraded to the latest release yesterday, which perfectly lines up with when the error started occurring for you. I've shipped a fix to their repo: rrweb-io/rrweb#1445 and added a patch to our own repo PostHog/posthog-js#1125 Thanks for reporting this 🙏 |
Thanks @daibhin for the speedy fix! Happy to help 😄 Should I go ahead and close issue or wait until the fix is deployed? |
Let's wait until the fix gets out and you're confident that the issue has been fixed for you |
Hey @daibhin, when can we expect this fix to be merged and deployed? Can we get a ETA on this. Thanks |
@bookee-sid I just merged the change. I should be live in the next hour or so. It will require that you upgrade your PostHog SDK to the latest version |
Thanks again @daibhin for the quick fix! Just confirmed that the error is no longer occurring so I will go ahead and close this issue. Have a good one! |
Amazing, thanks @dehli for confirming |
Bug description
Yesterday we started getting an error in our app triggered from Posthog hosted code:
Cannot read properties of null (reading 'tagName')
I was able to debug that it's related to user session recordings and disabling them prevents the error from showing more.
I have left it enabled on our dev instance so it can be more easily debugged.
How to reproduce
<redacted>
Environment
Additional context
The text was updated successfully, but these errors were encountered: