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

"interface address subnet should be /32 for IPv4 and /128 for IPv6" #82

Closed
JohanAR opened this issue Aug 9, 2023 · 5 comments · Fixed by #86
Closed

"interface address subnet should be /32 for IPv4 and /128 for IPv6" #82

JohanAR opened this issue Aug 9, 2023 · 5 comments · Fixed by #86

Comments

@JohanAR
Copy link
Contributor

JohanAR commented Aug 9, 2023

Is this check correct? The wireguard conf file that I've received from my VPN provider has subnets /24 for IPv4 and /64 for IPv6, and it works without issues with wg-quick.

In addition, the example wg.conf on ArchWiki also uses /24 and /64.

edit: I tried commenting out this line in config.go and everything seems to work with my VPN conf file.

@kubrickfr
Copy link
Contributor

I think it should work either way, this check is probably not necessary. For a single client (as opposed to a router), /32 or 128 makes "more sense" though, but in the end it should not matter.

@JohanAR
Copy link
Contributor Author

JohanAR commented Aug 16, 2023

Thanks! If wireproxy would be made more permissive, should this check be changed to something else, or could it be removed entirely?

@kubrickfr
Copy link
Contributor

I think it could be removed entirely, what do you think @pufferffish ?

@LittleVulpix
Copy link

@pufferffish , I know this may be asking a lot, but this one is actually a big help for my current setup. Would it be possible to release/build the binaries including this fix? I'm on windows and so I'm not exactly sure how I'd go around compiling or cross-compiling this.... I would be really grateful.

Thank you very much.

@kubrickfr
Copy link
Contributor

@LittleVulpix you really shouldn't need a new build. Just choose your IP and sticka /32 at the end of it. It should work just the same.

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 a pull request may close this issue.

3 participants