1533584 - Enable capping the maximum life-time of client-side cookies to seven days on Nightly

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, enhancement)

Description

[(no longer active)]

This bug is filed to enable the feature landed in bug 1529836 for seven days on Nightly. Sending an intent to implement to dev-platform now...

Comment 1

[(no longer active)]

Attachment: Bug 1533584 - Enable capping the maximum life-time of client-side cookies to seven days on Nightly;

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/10c036ba7c48
Enable capping the maximum life-time of client-side cookies to seven days on Nightly; r=baku

Comment 3

[Cosmin Sabou [:CosminS]]

Push with failures:
http://tinyurl.com/y6rl86lo
http://tinyurl.com/y6ydkpe2

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=235566390&repo=autoland&lineNumber=11819
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=235561003&repo=autoland&lineNumber=60642

Backout link: https://hg.mozilla.org/integration/autoland/rev/b93d00aab64923e7dbcebd06f422c50f4887e888

[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - TEST-UNEXPECTED-FAIL | /webdriver/tests/get_named_cookie/get.py | test_get_named_cookie - AssertionError: assert 'Fri, 29 Mar 2019 21:32:38' == 'Sat, 21 Mar 2020 21:32:38'
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - session = <Session 905320a4-f9a0-4e3c-9faf-53dc4252dbcf>
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - url = <function url at 0x7fef227bc8c0>
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO -
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - def test_get_named_cookie(session, url):
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - session.url = url("/common/blank.html")
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - clear_all_cookies(session)
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO -
[task 2019-03-22T21:32:38.914Z] 21:32:38 INFO - # same formatting as Date.toUTCString() in javascript
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO - utc_string_format = "%a, %d %b %Y %H:%M:%S"
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO - a_year_from_now = (datetime.utcnow() + timedelta(days=365)).strftime(utc_string_format)
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO - session.execute_script("document.cookie = 'foo=bar;expires=%s'" % a_year_from_now)
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO -
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO - result = get_named_cookie(session, "foo")
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO - cookie = assert_success(result)
[task 2019-03-22T21:32:38.915Z] 21:32:38 INFO - assert isinstance(cookie, dict)
[task 2019-03-22T21:32:38.916Z] 21:32:38 INFO -
[task 2019-03-22T21:32:38.916Z] 21:32:38 INFO - assert "name" in cookie
[task 2019-03-22T21:32:38.916Z] 21:32:38 INFO - assert isinstance(cookie["name"], basestring)
[task 2019-03-22T21:32:38.916Z] 21:32:38 INFO - assert "value" in cookie
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - assert isinstance(cookie["value"], basestring)
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - assert "expiry" in cookie
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - assert isinstance(cookie["expiry"], (int, long))
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO -
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - assert cookie["name"] == "foo"
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - assert cookie["value"] == "bar"
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - # convert from seconds since epoch
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - > assert datetime.utcfromtimestamp(
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - cookie["expiry"]).strftime(utc_string_format) == a_year_from_now
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - E AssertionError: assert 'Fri, 29 Mar 2019 21:32:38' == 'Sat, 21 Mar 2020 21:32:38'
[task 2019-03-22T21:32:38.917Z] 21:32:38 INFO - E - Fri, 29 Mar 2019 21:32:38
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - E ? ^^^ ^ ^^
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - E + Sat, 21 Mar 2020 21:32:38
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - E ? ^^^ ^ ^^
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO -
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - a_year_from_now = 'Sat, 21 Mar 2020 21:32:38'
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - cookie = {'domain': 'web-platform.test', 'expiry': 1553895158, 'httpOnly': False, 'name': 'foo', ...}
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - result = <Responsetatus=200 body={"value": {"domain": "web-platform.test", "name": "foo..."expiry": 1553895158, "path": "/common/", "httpOnly": false, "secure": false}}>
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - session = <Session 905320a4-f9a0-4e3c-9faf-53dc4252dbcf>
[task 2019-03-22T21:32:38.918Z] 21:32:38 INFO - url = <function url at 0x7fef227bc8c0>
[task 2019-03-22T21:32:38.919Z] 21:32:38 INFO - utc_string_format = '%a, %d %b %Y %H:%M:%S'

Comment 4

[Peter Van der Beken [:peterv]]

https://hg.mozilla.org/projects/ash/rev/10c036ba7c48082c3d2b93802384b8c756e82a73
Bug 1533584 - Enable capping the maximum life-time of client-side cookies to seven days on Nightly; r=baku

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cf5234d6c121
Enable capping the maximum life-time of client-side cookies to seven days on Nightly; r=baku

Comment 6

[Dorel Luca [:dluca]]

Backed out changeset cf5234d6c121 (bug 1533584) for Devtools falures in devtools/client/storage/test/browser_storage_values.js. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=237875667&repo=autoland&lineNumber=33236

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=cf5234d6c1212b9f5fa3f7030192a3a9066639c0

Backout:
https://hg.mozilla.org/integration/autoland/rev/d5bbc77e7a33b59edfc7d7885b7599c782da9b9d

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fa00c80a5c81
Enable capping the maximum life-time of client-side cookies to seven days on Nightly; r=baku

Comment 8

[Andreea Pavel [:apavel]]

Backed out for failing get_named_cookie/get.py and browser_storage_listings.js

Push with failures https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&selectedJob=237938040&revision=fa00c80a5c818e1d63b821b0524193a24b2f6627

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=237938617&repo=autoland&lineNumber=78680

Backout: https://hg.mozilla.org/integration/autoland/rev/1044c18486aa6239226963d7b83abb39e9f029b0

Backed out for failing dt at browser_storage_listings.js

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&selectedJob=237907427&searchStr=devtools&revision=fa00c80a5c818e1d63b821b0524193a24b2f6627

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=237939086&repo=autoland&lineNumber=7905

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dd68a6245f61
Enable capping the maximum life-time of client-side cookies to seven days on Nightly; r=baku

Comment 10

[(no longer active)]

Attachment: Bug 1533584 - Part 2: Make sure browser_storage_listings.js does not depend on the value of the privacy.documentCookies.maxage pref

Landed on a CLOSED TREE

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/98064c475d2f
Part 2: Make sure browser_storage_listings.js does not depend on the value of the privacy.documentCookies.maxage pref

Comment 12

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/dd68a6245f61
https://hg.mozilla.org/mozilla-central/rev/98064c475d2f