bug in nodriver browser.cookies.set_all(..)

[bluemangofunk]

Hi, I've tried to find a way to let you know on the nodriver site, but issues / pull requests limited to collaborators only.

At the end of the set_all function there is the code

cookies = await connection.send(cdp.storage.get_cookies())

As the cookies are passed in as a parameter cookies, this wipes the cookies as passed in

Solutions:

Change the above to

await connection.send(cdp.storage.get_cookies())

although I don't know what it does, for me and how I need it, it works equally to just remove the line :-)

Hoping this will help someone