feat(vpn-port-forwarding): make documentation up-to-date#134
Merged
Conversation
This was referenced Feb 1, 2025
qdm12
reviewed
Nov 13, 2025
qdm12
previously approved these changes
Nov 13, 2025
qdm12
left a comment
qdm12
left a comment
Owner
There was a problem hiding this comment.
Nice thanks for the various fixes!
qdm12
reviewed
Nov 13, 2025
Owner
|
Great work, just next time feel free to separate your PR (i.e. one PR per bullet point). It's easier to manage for me 😉 |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
I was setting up VPN port forwarding for my home lab recently, and observed many issues with the docs. I ultimately succeeded, but the information I needed was scattered between many files, issues in different repositories, and ultimately I had to read the source code of the implementation to understand what's going wrong.
In this PR, I habe
added a link to upcoming Transmission command (the link there will need to be changed once the script is merged)VPN_PORT_FORWARDING_LISTENING_PORTdoes - addressing Bug: VPN_PORT_FORWARDING_LISTENING_PORT not working passteque/gluetun#2520documented gluetun's Wireguard setup quirk (you have to use IP address instead of domain name) - addressing Feature request: Support for DNS domains in WIREGUARD_ENDPOINT_IP and wg0.conf passteque/gluetun#2680