tailscale/control/controlknobs/controlknobs.go, branch josh/debug-flakeThe easiest, most secure way to use WireGuard and 2FAhttp://git.waynecole.info/tailscale/atom?h=josh%2Fdebug-flake2021-07-16T05:34:50Zcontrol/{controlknobs,controlclient}: simplify knobs API, fix controlclient crash2021-07-16T05:34:50ZBrad Fitzpatrickbradfitz@tailscale.com2021-07-16T05:34:50Zurn:sha1:171ec9f8f4e4b4751b5f07dbed6f01514597c1a0
From integration tests elsewhere:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x70 pc=0x845c9b]
goroutine 226 [running]:
tailscale.com/control/controlclient.(*Direct).sendMapRequest(0xc00053e1e0, 0x16670f0, 0xc000353780, 0xffffffffffffffff, 0xc0003e5f10, 0x0, 0x0)
/home/runner/go/pkg/mod/tailscale.com@v1.1.1-0.20210715222212-1bb6abc604c1/control/controlclient/direct.go:803 +0x19bb
tailscale.com/control/controlclient.(*Direct).PollNetMap(...)
/home/runner/go/pkg/mod/tailscale.com@v1.1.1-0.20210715222212-1bb6abc604c1/control/controlclient/direct.go:574
tailscale.com/control/controlclient.(*Auto).mapRoutine(0xc00052a1e0)
/home/runner/go/pkg/mod/tailscale.com@v1.1.1-0.20210715222212-1bb6abc604c1/control/controlclient/auto.go:464 +0x571
created by tailscale.com/control/controlclient.(*Auto).Start
/home/runner/go/pkg/mod/tailscale.com@v1.1.1-0.20210715222212-1bb6abc604c1/control/controlclient/auto.go:151 +0x65
exit status 2
Also remove types/opt.Bool API addition which is now unnecessary.
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
net/portmapper: add upnp port mapping2021-07-15T22:22:12Zjulianknodtjulianknodt@gmail.com2021-06-22T22:29:01Zurn:sha1:1bb6abc604c1c798ea3aa2ea539273f0967089d8
Add in UPnP portmapping, using goupnp library in order to get the UPnP client and run the
portmapping functions. This rips out anywhere where UPnP used to be in portmapping, and has a
flow separate from PMP and PCP.
RELNOTE=portmapper now supports UPnP mappings
Fixes #682
Updates #2109
Signed-off-by: julianknodt <julianknodt@gmail.com>