[nodriver] Wrong set_all

[boludoz]

Fixed func.

async def set_all(cookies_handler, cookies: List[cdp.network.CookieParam]):
    """
    set cookies

    :param cookies: list of cookies
    :type cookies:
    :return:
    :rtype:
    """
    connection = None
    for tab in cookies_handler._browser.tabs:
        if tab.closed:
            continue
        connection = tab
        break
    else:
        connection = cookies_handler._browser.connection
    await connection.send(cdp.storage.set_cookies(cookies))

[boludoz]

I have to update the fucking function manually in all my codes.

[joshuaseltzer]

@ultrafunkamsterdam would you consider pushing this fix? There are 2 open PRs (8 and 16) that address this issue and will fix it immediately once merged.

If I had to guess, the issue itself is a copy/paste mistake from the get_all function. The problem is fixed by simply removing line 785 in browser.py which is erroneously overriding the provided cookies with cookies already loaded into the browser which completely breaks the ability to set cookies.