Skip to content

net: Update test cookie expiration dates to one century in the future.#44341

Merged
TimvdLippe merged 1 commit into
servo:mainfrom
jdm:net-expire-fix
Apr 19, 2026
Merged

net: Update test cookie expiration dates to one century in the future.#44341
TimvdLippe merged 1 commit into
servo:mainfrom
jdm:net-expire-fix

Conversation

@jdm

@jdm jdm commented Apr 19, 2026

Copy link
Copy Markdown
Member

These dates were chosen 10 years ago, which I'm sure felt like a reasonably far-future date for a very young web engine. Surprise! We're still here, so we can be more bold this time.

Testing: Unit tests stop failing.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Apr 19, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Apr 19, 2026
@TimvdLippe TimvdLippe added this pull request to the merge queue Apr 19, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Apr 19, 2026
Merged via the queue into servo:main with commit 19354a5 Apr 19, 2026
34 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Apr 19, 2026
@oldherl

oldherl commented Apr 19, 2026

Copy link
Copy Markdown

Not a serious problem, but the weekdays are wrong. For example, 18-Apr-2127 is a Friday, not Sunday.

@jdm

jdm commented Apr 19, 2026

Copy link
Copy Markdown
Member Author

Yeah, I think the date parsing ignores the day name so it's not a concern.

@amcnamara

Copy link
Copy Markdown

100 years from now someone will be equally annoyed that you didn't just set it to 3026. I hope that future coder reads this PR and only sets it 100 years into their future so that we can keep the chain of minor annoyances going.

@lookfirst

Copy link
Copy Markdown

Why not mock the system time so that this is never an issue?

Something like this: https://github.com/museun/mock_instant

@ductaiii

Copy link
Copy Markdown

What if in 100 years, someone is still here to update it to 1000 years 🤣🤣

jschwe pushed a commit to jschwe/servo that referenced this pull request Apr 27, 2026
servo#44341)

These dates were chosen 10 years ago, which I'm sure felt like a
reasonably far-future date for a very young web engine. Surprise! We're
still here, so we can be more bold this time.

Testing: Unit tests stop failing.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>
(cherry picked from commit 19354a5)
jschwe pushed a commit to jschwe/servo that referenced this pull request Apr 27, 2026
…the future.

This backports servo#44341 to release/v0.1 to fix unit-test failures,
due to hardcoded expiration date.

-----------------------------------

These dates were chosen 10 years ago, which I'm sure felt like a
reasonably far-future date for a very young web engine. Surprise! We're
still here, so we can be more bold this time.

Testing: Unit tests stop failing.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>
(cherry picked from commit 19354a5)
jschwe pushed a commit that referenced this pull request Apr 28, 2026
…the future.

This backports #44341 to release/v0.1 to fix unit-test failures,
due to hardcoded expiration date.

-----------------------------------

These dates were chosen 10 years ago, which I'm sure felt like a
reasonably far-future date for a very young web engine. Surprise! We're
still here, so we can be more bold this time.

Testing: Unit tests stop failing.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>
(cherry picked from commit 19354a5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants