Use boot-time for sleep #23
Conversation
Boot-time: "This library reimplements std::time::Instant to use suspend-aware monotonic time if target system supports it. Otherwise it uses monotonic time or reexports std::time::Instant." Will be obsoleted when tokio-rs/tokio#3185 is resolved. Fixes redlib-org#22
|
Testing this on MacOS for the efficacy of the fix. |
|
Hasn't worked yet. Please wait until the next commit. |
|
Another alternative to this would be to simply loop and check if the time had elapsed... Checking every minute for the current time won't take any more than a minuscule fraction of CPU time, and would handle suspends gracefully without relying on platform primitives. It also avoids the problem of figuring out when the auth token is expired and handling it out-of-band. |
|
Actually, it DOES work! I've had an instance running for three a days now; no error. The previous error must be due to some other cause? Restarting the instance to double check. |
|
Going to close this for now - as per most recent comment in linked issue, this occurs sometimes completely unrelatedly to boot time not counting towards timer, so this fix isn't going to be enough - we need to catch auth errors and force a refresh (which will fix this problem coincidentally). |
Boot-time: "This library reimplements std::time::Instant to use suspend-aware monotonic time if target system supports it.
Otherwise it uses monotonic time or reexports std::time::Instant."
Will be obsoleted when tokio-rs/tokio#3185 is resolved.
Fixes #22