Verifying the type of node during mutation of attributes

[zhanseitaliev]

To mutate attributes, a node must be of type Element.
In my case, during the iteration of the attributes for the mutation, some nodes were of type Text.
A node with type Text does not have a setAttribute method.

Issue: #209

[Yuyz0112]

Hi @zhanseitaliev, thanks for the PR.

Could you provide some events for debugging? I think this should not happen because we only collect attributes changes on elements.

[zhanseitaliev]

Hi @Yuyz0112,

This error appears in some sites with different structures.
Demo player: link (You can see the error after 5.30 s)
Events: link

Thank you for your response!

[Yuyz0112]

@zhanseitaliev Thanks for the help.

I see this is a multi-page website and there are some navigations. Are you putting rrweb on every page to trigger a record? Or maybe you can show some demo code of your record logic.

[zhanseitaliev]

@Yuyz0112 Yes, I added rrweb to every page.

import { record } from 'rrweb';

let events = [];

record({
    emit(event) {
        events.push(event);
    },
});

function save() {

    if(events.length > 0) {
        const body = JSON.stringify({ events });
        events = [];
        fetch('https://visor.shopter.tech/api/sessions/', {
            method: 'POST',
            headers: {
                'Content-Type': 'application/json'
            },
            body,
        })
    }

}

setInterval(save, 5 * 1000);

[Yuyz0112]

The events cause this problem is a little wired.

It looks like https://charlottelowe.co.uk/collections/photo-jewellery has more than one rrweb instance to record the page.

And I have tried using the REPL tool to record this page and things work well.
Is it possible to reproduce this now? (I see the events are logged on May 8)

[Yuyz0112]

Close due to un-reproducible. Feel free to reopen if the problem still happens.