Clarify peer configuration requirements for NAT-to-NAT communication in WireGuard #100
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.
Problem
The documentation previously stated that NAT-ed nodes should not define each other as peers outside of the public server config, suggesting that the relay server's catchall
AllowedIPs = 192.0.2.1/24route would handle all communication between NAT-ed clients. This guidance was misleading and would prevent NAT-ed peers from communicating with each other.Why this doesn't work:
WireGuard uses end-to-end encryption where each peer must have the public key of every peer it communicates with to perform cryptographic operations (handshakes, encryption, and authentication). The relay server only forwards encrypted packets at the network layer—it does not decrypt and re-encrypt traffic on behalf of clients.
If node A (behind NAT) tries to send traffic to node B (also behind NAT), but B isn't defined as a peer in A's config with its public key, WireGuard cannot encrypt the packets for B, and no traffic will flow.
Solution
This PR updates the documentation to:
Distinguish between two configuration scenarios:
Clarify peer definition requirements:
PublicKeyandAllowedIPsfor communicationEndpointis omitted for NAT-ed peers since they're not directly reachableAdd concrete examples:
Changes
[Peer]section to present both configuration scenarios clearlyTechnical Accuracy
These changes correctly reflect WireGuard's cryptographic architecture:
AllowedIPsenables routing; peer definitions enable encryptionImpact
This clarification will help users:
Fixes issue raised in #[issue_number] about clarification needed for peer configuration with NAT-ed nodes.
Original prompt
Fixes #95
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.