You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
tailscale/net
Denton Gentry ebc630c6c0 net/interfaces: also allow link-local for AzureAppServices.
In May 2021, Azure App Services used 172.16.x.x addresses:
```
10: eth0@if11: <BROADCAST,MULTICAST,UP,LOWER_UP,M-DOWN> mtu 1500 qdisc noqueue state UP
    link/ether 02:42:ac:10:01:03 brd ff:ff:ff:ff:ff:ff
    inet 172.16.1.3/24 brd 172.16.1.255 scope global eth0
       valid_lft forever preferred_lft forever
```

Now it uses link-local:
```
2: eth0@if6: <BROADCAST,MULTICAST,UP,LOWER_UP,M-DOWN> mtu 1500 qdisc noqueue state UP
    link/ether 8a:30:1f:50:1d:23 brd ff:ff:ff:ff:ff:ff
    inet 169.254.129.3/24 brd 169.254.129.255 scope global eth0
       valid_lft forever preferred_lft forever
```

This is reasonable for them to choose to do, it just broke the handling in net/interfaces.

This PR proposes to:
1. Always allow link-local in LocalAddresses() if we have no better
   address available.
2. Continue to make isUsableV4() conditional on an environment we know
   requires it.

I don't love the idea of having to discover these environments one by
one, but I don't understand the consequences of making isUsableV4()
return true unconditionally. It makes isUsableV4() essentially always
return true and perform no function.

Fixes https://github.com/tailscale/tailscale/issues/7603

Signed-off-by: Denton Gentry <dgentry@tailscale.com>
2 years ago
..
connstats net/connstats: fix ticker in NewStatistics (#7225) 2 years ago
dns various: pass logger.Logf through to more places 2 years ago
dnscache tstest: add method to Replace values for tests 2 years ago
dnsfallback various: pass logger.Logf through to more places 2 years ago
flowtrack all: update copyright and license headers 2 years ago
interfaces net/interfaces: also allow link-local for AzureAppServices. 2 years ago
memnet net/memnet: rename from net/nettest 2 years ago
netaddr all: update copyright and license headers 2 years ago
netcheck sockstats: switch label to enum 2 years ago
neterror all: update copyright and license headers 2 years ago
netknob all: update copyright and license headers 2 years ago
netns net/interfaces, net/netns: add node attributes to control default interface getting and binding 2 years ago
netstat net/{netns,netstat}: use new x/sys/cpu.IsBigEndian 2 years ago
netutil all: replace /kb/ links with /s/ equivalents 2 years ago
packet all: update copyright and license headers 2 years ago
ping all: update copyright and license headers 2 years ago
portmapper sockstats: switch label to enum 2 years ago
proxymux all: update copyright and license headers 2 years ago
routetable net/routetable: include unknown flags in the routetable doctor output 2 years ago
socks5 net/socks5: add password auth support 2 years ago
sockstats sockstats: remove per-interface stats from Get 2 years ago
speedtest all: update copyright and license headers 2 years ago
stun all: use Go 1.20's bytes.Clone 2 years ago
tlsdial health, net/tlsdial: add healthcheck for self-signed cert 2 years ago
tsaddr all: replace /kb/ links with /s/ equivalents 2 years ago
tsdial all: update copyright and license headers 2 years ago
tshttpproxy all: use tstest.Replace more 2 years ago
tstun wgengine: start logging DISCO frames to pcap stream 2 years ago
wsconn all: update copyright and license headers 2 years ago