Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] HeartBeat Support(NAT Support) #30

Open
Handsome1080P opened this issue Dec 4, 2021 · 5 comments · May be fixed by #93
Open

[Feature Request] HeartBeat Support(NAT Support) #30

Handsome1080P opened this issue Dec 4, 2021 · 5 comments · May be fixed by #93
Labels
enhancement New feature or request

Comments

@Handsome1080P
Copy link

Feature Request:

for nat,need hearbeat or keepalive

@dndx dndx added the enhancement New feature or request label Dec 4, 2021
@dndx
Copy link
Owner

dndx commented Dec 4, 2021

This is something I considered, however, I didn't add it due to the consideration that UDP protocols like WireGuard already have PersistentKeepalive which would keep the connections open, so I did not found TCP keepalive that important to add in the short term.

@Handsome1080P
Copy link
Author

Handsome1080P commented Dec 4, 2021

Yup,wg has the keepalive setting,but other udp tunnels not have one.Just follow ur priority.Thx

@kckm
Copy link

kckm commented Apr 12, 2022

I think let those udp tunnels that don't support keepalive join the heartbeat feature to be right.

@andreadaoud
Copy link

I think keep-alive is needed. If keep-alive packet is not received in-time, we must treat connection as broken, and re-establish TCP connection. WireGuard PersistentKeepalive cannot achieve this.

@Ginurx
Copy link

Ginurx commented Jul 19, 2024

Yup,wg has the keepalive setting,but other udp tunnels not have one.Just follow ur priority.Thx

Your issue can be temporarily addressed by setting a cron job which pings your server using nmap UDP scan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants