feat: add MTU option #173
Closed
feat: add MTU option #173
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| @@ -85,6 +86,7 @@ func (cmd *servecmd) Run() { | |||
| logrus.Infof("starting wireguard server on 0.0.0.0:%d", conf.WireGuard.Port) | |||
|
|
|||
| wgconfig := &wgembed.ConfigFile{ | |||
| // TODO: add MTU config | |||
There was a problem hiding this comment.
This can be done at a later time, what is more important is the client configuration generated by the admin-panel.
| @@ -60,6 +60,7 @@ export class AddDevice extends React.Component<Props> { | |||
| PrivateKey = ${privateKey} | |||
| Address = ${device.address} | |||
| ${info.dnsEnabled && `DNS = ${info.dnsAddress}`} | |||
| MTU = ${info.mtu} | |||
There was a problem hiding this comment.
This is the main component that fixes the issue.
DasSkelett
pushed a commit
to DasSkelett/wg-access-server
that referenced
this pull request
May 16, 2022
…n/website/material-ui/icons-4.11.3 Bump @material-ui/icons from 4.11.2 to 4.11.3 in /website
|
Do you want to add your PR to the active fork? @alexandre-abrioux https://github.com/freifunkMUC/wg-access-server |
|
@awlx Sure! thanks |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi! First of all I would like to thank you for maintaining this very cool project. It makes it so easy to set-up a VPN and has been working like a charm for us.
Context
If you're hosting your Wireguard server on Google Cloud platform you will encounter issues when accessing services protected by TLS or SSL, even more so if your internet service provider is using IPV6. This is due to the fact that GCP has a MTU allowance of 1460 bytes, which is too low for packets encrypted by Wireguard with its default MTU of 1420.
To solve this users have to manually edit their Wireguard configuration file and add the following line under the
[Interface]section:See several sources talking about this issue:
Changes
I tried to start a PR in order to add a new
WG_WIREGUARD_MTUconfiguration to resolve this. However I have no experience at all in Go, so I'm just spitballing here in the changes I made. The gRPC code has been manually edited because I was not able to successfully run the command./codegen.sh.This PR was not tested. It looks complete but please consider this as a work in progress. If someone could take over to bring this feature to the finish line, it would be awesome!
Thanks!
PS: this would fix #112