synchronized-promise icon indicating copy to clipboard operation
synchronized-promise copied to clipboard

Change default timeout to infinity

Open Yukaii opened this issue 7 years ago • 2 comments

時間控制算是 optional 的

Yukaii avatar Feb 20 '18 16:02 Yukaii

@Yukaii In my use case, I was polling for a dependent service. Until that service comes up my server will have to wait forever. I would love to see the options parameter accept {timeouts: Number.POSITIVE_INFINITY} If you accept the pull request I can make that happen. code that error out if timeouts equal Infinity : image

https://github.com/Yukaii/synchronized-promise/blob/master/lib/index.js#L34

SuperCipher avatar Mar 08 '21 09:03 SuperCipher

Hi @SuperCipher, feel free to open a PR!

Yukaii avatar Mar 08 '21 11:03 Yukaii