Close inactive connections

Open WyriHaximus opened this issue 4 years ago • 1 comments

This new middleware introduces a timeout of closing inactive connections between connections after a configured amount of seconds.

This builds on top of #405 and partially on #422

This PR very specifically doesn't deal with inactivity while handling requests, which will be proposed in a different PR.

Aug 21 '21 20:08 WyriHaximus

@WyriHaximus Thanks for the update!

Can you provide a quick update here, what's your plan for this feature with regards to #425? We've briefly discussed (off channel) that this should eventually send a 408 Request Timeout response for the initial connection but not for keep-alive connections that send no following request. I agree breaking this up into smaller PRs can be useful, but it looks like this might also work in a single PR, so I'll leave this up to you. Let me know if there's anything I can help with and/or if you want me to take over from here +1

@clue Will fold it into a single PR, and will be adding the 408 next :+1:

Nov 09 '21 20:11 WyriHaximus