moby / vpnkit

A toolkit for embedding VPN capabilities in your application
Apache License 2.0
1.09k stars 182 forks source link

Ignore a dup SYN for older connections as well #546

Closed yamt closed 2 years ago

yamt commented 3 years ago

I occasionally observed stalls because of this when using NuttX sim as a client.

djs55 commented 3 years ago

@yamt thanks for the patch! I think I might have seen a similar problem.

I'll test it and let you know what I find. Unfortunately the CI isn't fully enabled on this repo at the moment.

yamt commented 2 years ago

i applied this to our private ci two months ago. i haven't noticed any issues since then. before applying this patch, it was occasionally failing. (once a few days or so)

djs55 commented 2 years ago

@yamt thanks for the update. I'll merge the patch now.