t3hmrman / situwaition

⏲ situwaition is a Rust utility library that waits for conditions
https://crates.io/crates/situwaition
MIT License
7 stars 0 forks source link

Timeouts may not be upheld with long sync check function #4

Closed t3hmrman closed 1 year ago

t3hmrman commented 1 year ago

If given a sync function that resolves after a long interval (ex. 10 minutes), the SyncWaiter will blow past it's timeout.

Thanks to TroyDota on Reddit for gesturing towards the easiest mitigation -- spawning a thread and checking for completion.

It's a bit of an edge case (most checks will be short) but timeouts not being honored is a bug.