heydryft / tlsclient.js

An axios based wrapper for bogdanfinn/tls-client based on ffi-rs for unparalleled performance and usability.
28 stars 9 forks source link

Responses don't return all 'set-cookie' headers #1

Open bradryannelson opened 8 months ago

bradryannelson commented 8 months ago

it's possible for there to be multiple headers with the same key ('set-cookie' for example). If there are multiple, it will only return 1.

ex: 'Set-Cookie': 'cookieA=value; path=/; secure; HttpOnly; SameSite=None', 'Set-Cookie': 'cookieB=value; path=/; secure; HttpOnly; SameSite=None', 'Set-Cookie': 'cookieC=value; path=/; secure; HttpOnly; SameSite=None',

will only return 'Set-Cookie': 'cookieA=value; path=/; secure; HttpOnly; SameSite=None',

heydryft commented 8 months ago

I think this was fixed with the latest merge, can you check?

preayham commented 7 months ago

hey, same issue still only returning first cookie. I have also tried using tough-cookie with this using the axios cookie jar wrapper and it doesnt see the set-cookie header.