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

Fix of issue of modification on active tunnel #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

suquant
Copy link

@suquant suquant commented Dec 7, 2021

Signed-off-by: George Kutsurua [email protected]

Hello,

This pr contains a fix for the case when the tunnels manager modifies the active tunnel.

Use case:
Once the active tunnel is modified it is sometimes disconnected and does not connect back again, sometimes it is stuck in "waiting" status.

Description of the issue:
When we are modifying the active tunnel there process tunnel's status programmatically changes its status to "restarting" and later this status processes properly (connect back disconnected tunnel). But problem is that once the tunnel changes its configuration "reload()" is triggered in "startObservingTunnelConfigurations" function and it is replacing tunnels, so programmatically set status never reaches. It works sometimes well because of "reload" function executes after status processing and sometimes not, depends when reload start's its execution.

@suquant suquant force-pushed the f/fix-tunnel-status-proccessing branch from 256f43b to d08ee36 Compare December 7, 2021 11:44
@s4w4j4
Copy link

s4w4j4 commented Mar 25, 2022

Hello, when the IP address of the server changes, the client will lose the VPN connection and must restart the client to recover. Can you help solve this problem? thank you

kar pushed a commit to blokadaorg/wireguard-apple that referenced this pull request Dec 16, 2022
@zx2c4-bot zx2c4-bot force-pushed the master branch 2 times, most recently from c37881b to 2fec12a Compare February 15, 2023 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants