-
Notifications
You must be signed in to change notification settings - Fork 134
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
InvalidStateError: Invalid resize dimensions #1150
Comments
@daibhin is this resolved in recent changes? |
I don't think because we haven't applied any fixes to rrweb on the SDK since upgrading to I'm not sure I've seen that particular error before. It seems similar to rrweb-io/rrweb#1422 but difficult to tell without knowing what caused it to throw. @jmillxyz did the error lead to a particular line in the source code? Alternatively I'd be happy to jump in and debug if this was easily reproducible on your site |
@daibhin it seems easily reproducible to me -- our site is mainshares.com. On macOS 14.4.1, Safari 17.4.1 I see On macOS 14.4.1, Chrome 124.0.6367.119 either I misspoke or since v1.130.1 the error is slightly different, I now see |
Thanks for the context! It looks like the open-source package we use to capture DOM mutations isn't catching a specific error thrown by For your specific case @jmillxyz - I noticed that there is a canvas element on your homepage but it doesn't seem to be rendered anywhere. Could it possibly be remove from the DOM? That would fix the exception for you |
Ah, this makes sense -- we use I tested removing topbar locally and no longer see the error. Do you have any suggestions on how to handle this situation? I can imagine it probably affects anyone else with this library. |
Hi @daibhin, a bit more context -- our prod posthog project has the "Capture canvas elements" setting turned on. My change to delete For now I'm going to turn off this setting, as we don't have many canvas elements that need to be recorded. It seems like there is still some kind of underlying issue when canvas elements are removed, though |
chiming in here - our app uses canvases extensively (it's a game), and this issue creates a TON of noise in our logs. would love to see it fixed.
actually that issue must not have fixed it, since it was included in rrweb alpha 13, which is already used by posthog, and we still have the issue |
We recently began using
posthog-js
session recording via the NPM library. Recently we've gotten thisInvalidStateError: Invalid resize dimensions
with the following backtrace, which seems to be related torrweb
:We also saw this with
[email protected]
:It seems to happen to at least Safari and Chrome. If there's more data I can provide to help triage please let me know!
The text was updated successfully, but these errors were encountered: