| Age | Commit message (Collapse) | Author | Files | Lines |
|
Fixes #1214
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
|
|
Updates #1187
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Fixes a regression from e970ed09951a that wasn't covered by tests
in this repo. (Our end-to-end tests in another repo caught this.)
Updates #1204
|
|
Signed-off-by: Sonia Appasamy <sonia@tailscale.com>
Consolidates the node display name logic from each of the clients into
tailcfg.Node. UI clients can use these names directly, rather than computing
them independently.
|
|
Fixes #1208
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
Also don't log.Fatalf in a function returning an error.
Fixes #1204
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
|
|
|
|
Updates #1187
|
|
No functional change. Make a future diff easier to read.
|
|
Updates #1187
|
|
it from controlclient (#1191)
Signed-off-by: Sonia Appasamy <sonia@tailscale.com>
|
|
On Windows, configureInterface starts a goroutine reconfiguring the
Windows firewall.
But if configureInterface fails later, that goroutine kept running and
likely failing forever, spamming logs. Make it stop quietly if its
launching goroutine filed.
|
|
Should be equivalent to the license before, but compatible with the library Github uses to detect the license for the project's metadata: https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/licensing-a-repository#detecting-a-license
Signed-off-by: Andrey Petrov <andrey.petrov@shazow.net>
|
|
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
|
|
Suggested by Avery earlier. Ends up fixing bug in "tailscale status" when
MagicDNS if off too:
https://forum.tailscale.com/t/1-3-293-is-released-a-1-4-0-pre-release/349/11?u=bradfitz
|
|
Fixes #1192 (regression)
Fixes #1193 (old bug)
|
|
Not a problem (yet). But should be consistent with other places that support both
types of drops.
|
|
|
|
|
|
Rewrite log lines on the fly, based on the set of known peers.
This enables us to use upstream wireguard-go logging,
but maintain the Tailscale-style peer public key identifiers
that the rest of our systems (and people) expect.
Fixes #1183
Signed-off-by: Josh Bleecher Snyder <josh@tailscale.com>
|
|
This lets "tailscale ping $NAME" work even if MagicDNS is off, letting you
ping a name that shows up in "tailscale status".
More user friendly.
|
|
Reduces background CPU & network.
Updates #1034
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
Continuation of earlier two umask changes,
5611f290eb118eddc256560eaaa69f509347b4de and
d6e9fb1df0fd67d08065c2277e7c4f4a82b7930f.
This change mostly affects us, running tailscaled as root by hand (wit
a umask of 0077), not under systemd. End users running tailscaled
under systemd won't have a umask.
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Signed-off-by: Steve Coffman <steve@khanacademy.org>
|
|
Also, don't try to use IPv6 LinkLocalUnicast addresses for now. Like endpoints
exchanged with control, we share them but don't yet use them.
Updates #1172
|
|
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
|
|
c8c493f3d9bf925e9459236bf1ecea823be6f825 made it always say
`created=false` which scared me when I saw it, as that would've implied
things were broken much worse. Fortunately the logging was just wrong.
|
|
|
|
Fixes #1172
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
This adds a more wire-efficient way of updating peers' Node.LastSeen times.
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Updates #1172
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
DstToString is used in two places in wireguard-go: Logging and uapi.
We are switching to use uapi for wireguard-go config.
To preserve existing behavior, we need the full set of addrs.
And for logging, having the full set of addrs seems useful.
(The Addrs method itself is slated for removal. When that happens,
the implementation will move to DstToString.)
Signed-off-by: Josh Bleecher Snyder <josh@tailscale.com>
|
|
To save CPU and wakeups, don't run the DERP cleanup timer regularly
unless there is a non-home DERP connection open.
Also eliminates the goroutine, moving to a time.AfterFunc.
Updates #1034
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
|
|
connection is closed"
This reverts commit 08baa17d9a785635891c3462d01e601759b8b8b6.
It caused deadlocks due to lock ordering violations.
It was not the right fix, and thus should simply be reverted
while we look for the right fix (if we haven't already found it
in the interim; we've fixed other logging-after-test issues).
Fixes #1161
|
|
|
|
The 'ok' from 'ipp, ok :=' above was the result parameter ok. Whoops.
|
|
name old time/op new time/op delta
ReceiveFrom-4 21.8µs ± 2% 20.9µs ± 2% -4.27% (p=0.000 n=10+10)
Updates #414
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
It's about to become a no-op in control.
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
name old time/op new time/op delta
ReceiveFrom-4 35.8µs ± 3% 21.9µs ± 5% -38.92% (p=0.008 n=5+5)
Fixes #1145
Updates #414
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Signed-off-by: David Anderson <danderson@tailscale.com>
|
|
|