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

Flags refactoring + custom MTU #105

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

Conversation

cheina97
Copy link

@cheina97 cheina97 commented Jun 24, 2024

This PR refactored flags using pflag packages.

The reason for this PR is to enable adding new flags to customize the wireguard parameters.
For example, the MTU has been added to the flags in this PR.

It also introduces the possibility to use the --help flag to information about the usage of wireguard-go:

Usage: ./wireguard-go [flags] <interface-name>
      --foreground   Remain in the foreground
      --mtu int      Set the MTU of the device (default 1420)
  -v, --version      Print the version number and exit
pflag: help requested 

@cheina97 cheina97 force-pushed the frc/flags branch 2 times, most recently from 2b5cc16 to 865d92d Compare June 24, 2024 09:35
Signed-off-by: Francesco Cheinasso <[email protected]>
Signed-off-by: Francesco Cheinasso <[email protected]>
@cheina97
Copy link
Author

@zx2c4 how can I request a review?

@pronebird
Copy link

Can't you set mtu via wg wconfig?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants