search

ntop / n2n

Peer-to-peer VPN
GNU General Public License v3.0
6.05k stars 925 forks source link

Purgable fixes #1147

Closed hamishcoleman closed 8 months ago

hamishcoleman commented 10 months ago

Hopefully addressing #1146

codecov-commenter commented 10 months ago

Codecov Report

Patch has no changes to coverable lines.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files Changed Coverage
src/edge_management.c 0.00%
src/edge_utils.c 0.00%
src/management.c 0.00%
src/n2n.c 0.00%
src/sn_management.c 0.00%
src/sn_utils.c 0.00%

:loudspeaker: Thoughts on this report? Let us know!.

JimLee1996 commented 9 months ago

Hope this fix merged ASAP, as the reconnection stategy may fail due to uncorrect count of peers. https://github.com/ntop/n2n/blob/dev/src/edge_utils.c#L1622

sduensin commented 9 months ago

I've been using this since it was posted. It takes a bit longer than I'd like to purge a disconnected peer, but it works!