Open
Bug 1427807
Opened 7 years ago
Updated 2 years ago
WebDriver:GetCookies does not return cookies from frames
Categories
(Remote Protocol :: Marionette, defect, P3)
Tracking
(firefox57 wontfix, firefox58 wontfix, firefox59 wontfix, firefox60 wontfix, firefox61 wontfix, firefox62 wontfix, firefox63 wontfix, firefox64 wontfix, firefox65 fix-optional)
People
(Reporter: whimboo, Unassigned)
References
(Blocks 2 open bugs)
Details
(Keywords: regression)
Originally filed as https://github.com/mozilla/geckodriver/issues/1104
With bug 1371733 (fixed in Firefox 56) the cookies are collected via the parent process now. As such it seems that we have no longer access to cookies as defined in frames.
Here a simple Marionette test showing this problem (based on a reporter testcase):
def test(self):
self.marionette.navigate("https://x.zok.pw/geckodrivertest/outer.html")
frame = self.marionette.find_element(By.ID, "subframe")
self.marionette.switch_to_frame(frame)
self.assertIsNotNone(self.marionette.get_cookie("inner"))
Andreas, could you please have a look?
Flags: needinfo?(ato)
|
||
Comment 1•7 years ago
|
||
(In reply to Henrik Skupin (:whimboo) [mostly away Dec 11th - Jan
3rd] from comment #0)
> With bug 1371733 (fixed in Firefox 56) the cookies are collected
> via the parent process now. As such it seems that we have no
> longer access to cookies as defined in frames.
We always got the cookies from chrome space, but I’d bet we need to
use the hostname of the current frame (fetched from listener.js)
instead of the browser’s current URL when enumerating them.
Blocks: webdriver
Flags: needinfo?(ato)
Summary: Get cookies does not return cookies from frames since cookies are collected by the parent process → WebDriver:GetCookies does not return cookies from frames
|
||
Comment 2•7 years ago
|
||
Hi, I am the original reporter of this issue.
Is there any workaround that I can use in selenium test with geckodriver?
|
|
||
Updated•7 years ago
|
Priority: P3 → P2
Too late to fix in 59, but we can still take a patch in Nightly.
status-firefox60:
--- → fix-optional
status-firefox61:
--- → affected
status-firefox62:
--- → fix-optional
status-firefox63:
--- → fix-optional
|
||
Updated•6 years ago
|
Happy to take a patch in nightly; if it seems low risk enough please feel free to request uplift to 65 beta.
|
Reporter | |
Updated•4 years ago
|
Flags: needinfo?(dburns)
|
||
Updated•2 years ago
|
Severity: normal → S3
|
|
||
Updated•2 years ago
|
Product: Testing → Remote Protocol
|
||
Comment 6•2 years ago
|
||
Hi, is there any update on this ticket?
|
|
Reporter | |
Comment 7•2 years ago
|
||
Sorry, but not as of this point. This broken behavior is not high on our priority list. But if someone has interest to work on it, I'm happy to mentor.
|
|
Reporter | |
Updated•2 years ago
|
Priority: P2 → P3
|
|
Reporter | |
Comment 8•2 years ago
|
||
Meni.pablo as we spoke on Matrix I would be interested if you still want to help or if you wont have the time anymore. Thanks.
Flags: needinfo?(meni.pablo)
|
|
||
Comment 9•2 years ago
|
||
(In reply to Henrik Skupin [:whimboo][⌚️UTC+1] from comment #8)
Meni.pablo as we spoke on Matrix I would be interested if you still want to help or if you wont have the time anymore. Thanks.
Yes, I am still interested in this. I'll reach out to you on Matrix for the next steps
Flags: needinfo?(meni.pablo)
You need to log in
before you can comment on or make changes to this bug.
Description
•