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

Add node VPN details #668

Merged
merged 2 commits into from
Jul 15, 2024
Merged

Add node VPN details #668

merged 2 commits into from
Jul 15, 2024

Conversation

DavidePrincipi
Copy link
Member

  • Return the leader vpn endpoint and wg listen port
  • Return the worker vpn endpoint and guess the wg listen port
  • Ensure int parameters are converted safely: if keepalive is manually disabled, an error occurs

Refs NethServer/dev#6960

@DavidePrincipi DavidePrincipi self-assigned this Jul 12, 2024
github-actions bot pushed a commit that referenced this pull request Jul 12, 2024
@DavidePrincipi DavidePrincipi changed the base branch from main to feat-6960 July 12, 2024 14:22
- Return the leader vpn endpoint and the wg listen port
- Return the worker vpn endpoint, obtain the wg listen port from the
  endpoint. If no endpoint is set in Redis, fall back to wg runtime
  values.
- Ensure int parameters are converted safely: if keepalive is manually
  disabled, an error occurs
Copy link
Contributor

@andre8244 andre8244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a typo

github-actions bot pushed a commit that referenced this pull request Jul 15, 2024
@DavidePrincipi DavidePrincipi merged commit 0426b81 into feat-6960 Jul 15, 2024
3 checks passed
@DavidePrincipi DavidePrincipi deleted the feat-6960-2 branch July 15, 2024 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants