[meta] Network.getAllCookies
Categories
(Remote Protocol :: CDP, task)
Tracking
(Not tracked)
People
(Reporter: ato, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: meta, Whiteboard: [method=Network.getAllCookies])
Network.getCookies
was split into Network.getCookies
, which returns the cookies for the current URL, and Network.getAllCookies
, which returns all the cookies in storage, on 13th December of this year as part of 9349e8fc
.
Comment hidden (obsolete) |
Comment 2•4 years ago
|
||
Support for Network.getCookies
will land on bug 1590098 soon. Lets see which code we can re-use.
Reporter | ||
Comment 3•4 years ago
•
|
||
(In reply to Henrik Skupin (:whimboo) [⌚️UTC+1] from comment #1)
Note that this new behavior is only available in Canary builds of Chrome. A lot of users will still run an older version of Chrome matching the version of Puppeteer. So as long as there is no new version of Puppeteer out I wouldn't suggest to make that change specifically to
Network.getCookies()
which is basically backward incompatible.
We need to make a decision before we can do schema validation regarding which CDP version to target. I think I agree it makes sense to target the same CDP version used in Puppeteer.
Comment 4•4 years ago
|
||
Note the above discussion renders obsolete given that the before-mentioned feature got landed Dec 2016 and not 2019. So there is nothing which currently blocks us here implementing it.
Comment 5•4 years ago
|
||
There is basically Services.cookies.cookies
which holds the list of all the stored cookies. I assume that's what we have to return here.
Comment 6•4 years ago
|
||
With the landed work on bug 1637619 nothing more should be needed for this bug.
Assignee | ||
Updated•3 years ago
|
Description
•