From 80c94168ae250178c0a119b03d3f593c2ee10f4e Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Tue, 22 Dec 2020 14:48:24 -0800 Subject: [PATCH] wgengine: finish updating isTrimmablePeer I accidentally merged Dave's change with the XXXX DO NOT SUBMIT comment in it. --- wgengine/userspace.go | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/wgengine/userspace.go b/wgengine/userspace.go index 9064f524a..a0f9e074a 100644 --- a/wgengine/userspace.go +++ b/wgengine/userspace.go @@ -626,11 +626,9 @@ func forceFullWireguardConfig(numPeers int) bool { // We can only trim peers that both a) support discovery (because we // know who they are when we receive their data and don't need to rely // on wireguard-go figuring it out) and b) for implementation -// simplicity, have only one IP address (an IPv4 /32), which is the -// common case for most peers. Subnet router nodes will just always be -// created in the wireguard-go config. -// -// XXXXXXX DO NOT SUBMIT fix docstring +// simplicity, have only non-subnet AllowedIPs (an IPv4 /32 or IPv6 +// /128), which is the common case for most peers. Subnet router nodes +// will just always be created in the wireguard-go config. func isTrimmablePeer(p *wgcfg.Peer, numPeers int) bool { if forceFullWireguardConfig(numPeers) { return false