search

nodejs / undici

An HTTP/1.1 client, written from scratch for Node.js
https://nodejs.github.io/undici
MIT License
6.21k stars 541 forks source link

TIMEOUT_IDLE should include event loop lag #3553

Closed ronag closed 1 month ago

ronag commented 1 month ago

Tee TIMEOUT_IDLE state used by the keep alive timeout should include the event loop lag since it's tracking a remote servers time.

I think the recent fast timers event loop lag fixes has broken this. We should not use fast timers for this state.

Uzlopak commented 1 month ago

Provided PR #3554 ;)