Cannot read properties of undefined (in rrweb)

[jlowcs]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/react

SDK Version

9.10.1

Framework Version

React 18.3.1

Link to Sentry event

N/A

Reproduction Example/SDK Setup

No response

Steps to Reproduce

Hard to reproduce, but it seems to be related to a third-party utility that makes changes to the DOM.

It results in an error in the browser console pointing to a genAdds function, which I'm pretty sure is this line.

I think adding a check that n.childNodes is defined should do the trick:

      if (n.childNodes)
        n.childNodes.forEach((childN) => this.genAdds(childN));

Expected Result

No error in the console

Actual Result

error in the console

[andreiborza]

Hi, thanks for investigating and filing this! Would you like to contribute this and open up a PR? No pressure, we can also take care of that.

[mydea]

Hmm, technically speaking we should do el. hasChildNodes() before accessing the child nodes, apparently (see https://developer.mozilla.org/en-US/docs/Web/API/Node/childNodes#simple_usage).

I'll see if we can fix this easily...