<- Back
Comments (69)
- vocx2txBut still a kludge. Better: use something equivalent to Go's testing/synctest[0] package, which lets you write tests that run in a bubble where time is fixed and deterministic.[0] https://pkg.go.dev/testing/synctest
- andaiInteresting, from the title I thought it was intentional, as a "forced code review." Apparently not, but now I really like that idea!
- AlupisJust skimmed the PR, I'm sure the author knows more than I - but why hard code a date at all? Why not do something like `today + 1 year`?
- bombcarAny time constant will be exceeded someday.An impossibly short period of time after the heat death of the universe on a system that shouldn’t even exist: ERROR TIME_TEST FAILURE
- harikbA comment from the PR> Not a serious problem, but the weekdays are wrong. For example, 18-Apr-2127 is a Friday, not Sunday.There is now many magical dates to remember - 2126 ( I think PR was updated after that comment) and 2177. There is also 2028 also somewhere.
- samlinnferi had to plant a 10 year time bomb in our SAML SP certificate because AFAIK there is no other way to do it. It’s been 7 years since then. Dreading contacting all the IDPs and getting them to update the SAML config.
- db48xClassic!But before you judge the fix too hashly, I bet it’s just a quick and easy fix that will suffice while a proper fix (to avoid depending on external state) is written.
- kristofferR[flagged]
- dhosekOne of the comments:> Us, ten years after generating the certificate: "Who could have possibly foreseen that a computer science department would still be here ten years later."This was why there was a Y2K bug. Most of that code was written in the 80s, during the Reagan era. Nobody expected civilization to make it to the year 2000.