Closed
Bug 1879503
Opened 4 months ago
Closed 2 months ago
Update implementation of `getCookies` command to align with gecko default cookie behaviour
Categories
(Remote Protocol :: WebDriver BiDi, task, P2)
Remote Protocol
WebDriver BiDi
Tracking
(firefox126 fixed)
RESOLVED
FIXED
126 Branch
| Tracking | Status | |
|---|---|---|
| firefox126 | --- | fixed |
People
(Reporter: Sasha, Assigned: Sasha)
References
(Blocks 1 open bug)
Details
(Whiteboard: [webdriver:m11][wptsync upstream][webdriver:relnote])
Attachments
(2 files, 1 obsolete file)
In the scope of bug1854580 we added one test which verifies the output of getCookies command against the cookies which are sent with network request. To ensure the correctness of the getCookies command, we should add more tests like this, e.g. just for a top-level document, for a cross-origin fetch() (with credentials) inside a top-level document, for a cross-origin image, etc.
And we should also update the implementation of getCookies command to work with network.cookie.cookieBehavior = 5, which is a default behaviour, instead of it equals 0 (right now this value is set as pref for CDP).
|
Assignee | |
Updated•4 months ago
|
Priority: -- → P2
Whiteboard: [webdriver:backlog]
|
|
Assignee | |
Comment 1•4 months ago
|
||
|
||
Updated•4 months ago
|
Assignee: nobody → aborovova
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•4 months ago
|
Assignee: aborovova → nobody
Status: ASSIGNED → NEW
|
|
Assignee | |
Updated•4 months ago
|
Summary: Add more tests for `getCookies` command with network events → Update implementation of `getCookies` command to align with gecko default cookie behaviour
|
|
Assignee | |
Updated•3 months ago
|
Assignee: nobody → aborovova
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•3 months ago
|
||
|
|
Assignee | |
Comment 3•3 months ago
|
||
|
|
||
Updated•3 months ago
|
Attachment #9379982 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•3 months ago
|
Points: --- → 3
Whiteboard: [webdriver:backlog] → [webdriver:m10]
|
||
Updated•2 months ago
|
Whiteboard: [webdriver:m10] → [webdriver:m11]
Pushed by aborovova@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/275fdabfd6cd Do not send third-party cookies when it's not allowed by preferences. r=webdriver-reviewers,whimboo https://hg.mozilla.org/integration/autoland/rev/700fbf694ba4 [wdspec] Add tests for "storage.getCookies" command with network event. r=webdriver-reviewers,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/45643 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m11] → [webdriver:m11], [wptsync upstream]
Upstream PR was closed without merging
|
||
Comment 7•2 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/275fdabfd6cd
https://hg.mozilla.org/mozilla-central/rev/700fbf694ba4
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
status-firefox126:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
Upstream PR merged by moz-wptsync-bot
Upstream PR merged by moz-wptsync-bot
|
||
Updated•1 month ago
|
Whiteboard: [webdriver:m11], [wptsync upstream] → [webdriver:m11][wptsync upstream][webdriver:relnote]
You need to log in
before you can comment on or make changes to this bug.
Description
•