mozilla / geckodriver

WebDriver for Firefox
https://firefox-source-docs.mozilla.org/testing/geckodriver/
Mozilla Public License 2.0
7.12k stars 1.52k forks source link

Geckodriver caches expired cookies when interrogated BEFORE expiry #1918

Open luke-hill opened 3 years ago

luke-hill commented 3 years ago

System

Testcase

Add any cookie you desire, with a ttl of around 10-15 seconds from the current time.

Positive test

Failing test

Stacktrace

N/A

Trace-level log

N/A

whimboo commented 3 years ago

Could you please attach a trace-level log from geckodriver? Read more about reporting actionable bugs in our contribution guidelines.

luke-hill commented 3 years ago

Will attach (Warning it's 600+ lines for a tiny test)

luke-hill commented 3 years ago

webdriver.log

luke-hill commented 3 years ago

My gut is the length of the timestamp you are using in marionette is different to the length of the timestamp returned from the cookie.

Timestamp used by marionette is i.e. 1631002848625 which is 13 digits long and the timestamp in cookies is i.e. 1631002844 which is 10 digits long.

luke-hill commented 2 years ago

ping @whimboo