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

Skip auto-transport when transport available #1299

Merged
merged 6 commits into from
Jul 28, 2022
Merged

Skip auto-transport when transport available #1299

merged 6 commits into from
Jul 28, 2022

Conversation

mrpalide
Copy link
Contributor

Did you run make format && make check? Yes

Fixes #1297

Changes:

  • add a condition at the beginning of getRouteSetupHooks method (a.k.a. autotransport) to skip whole of process if a transport from visor to vpn-server be available.

How to test this PR:

  • make build
  • run a visor (in debug log-level), and run a vpn-server (you can use 036c0e89735c675abce702a3d46ad1bf736bd8236dbd734bfaf368959de2c2b967)
  • create manual transport in dmsg or sudph to vpn-server
  • trying to run vpn-client
  • should see this log on visor logs Debug [router]: Established transport exist. Type: dmsg, and no trying for new transport in order stcpr>sudph>dmsg
  • also vpn-client should running on exist transport, so check Routing tab of visor details

@jdknives jdknives merged commit dcd5825 into skycoin:develop Jul 28, 2022
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.

None yet

3 participants