synced/coder
1
0
Fork 0
mirror of https://github.com/coder/coder.git synced 2025-07-08 11:39:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: upgrade tailscale to fix STUN probes on dual stack (#10535)

Browse Source 
Fixes STUN probe issues on dual stack systems by incorporating https://github.com/coder/tailscale/pull/43
This commit is contained in:
Spike Curtis
2023-11-07 08:48:27 +04:00
committed by GitHub
parent 14925e71a7
commit c87deb868b
3 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
coderd/healthcheck/derphealth/derp_test.go
View File

@ -69,9 +69,9 @@ func TestDERP(t *testing.T) {
assert.True(t, node.CanExchangeMessages)
assert.NotEmpty(t, node.RoundTripPing)
assert.Len(t, node.ClientLogs, 2)
assert.Len(t, node.ClientLogs[0], 1)
assert.Len(t, node.ClientLogs[0], 3)
assert.Len(t, node.ClientErrs[0], 0)
assert.Len(t, node.ClientLogs[1], 1)
assert.Len(t, node.ClientLogs[1], 3)
assert.Len(t, node.ClientErrs[1], 0)
assert.False(t, node.STUN.Enabled)
@ -113,9 +113,11 @@ func TestDERP(t *testing.T) {
assert.True(t, node.CanExchangeMessages)
assert.NotEmpty(t, node.RoundTripPing)
assert.Len(t, node.ClientLogs, 2)
assert.Len(t, node.ClientLogs[0], 1)
// the exact number of logs depends on the certificates, which we don't control.
assert.GreaterOrEqual(t, len(node.ClientLogs[0]), 1)
assert.Len(t, node.ClientErrs[0], 0)
assert.Len(t, node.ClientLogs[1], 1)
// the exact number of logs depends on the certificates, which we don't control.
assert.GreaterOrEqual(t, len(node.ClientLogs[1]), 1)
assert.Len(t, node.ClientErrs[1], 0)
assert.True(t, node.STUN.Enabled)
@ -125,7 +127,7 @@ func TestDERP(t *testing.T) {
}
})
t.Run("ForceWebsockets", func(t *testing.T) {
t.Run("FailoverToWebsockets", func(t *testing.T) {
t.Parallel()
derpSrv := derp.NewServer(key.NewNode(), func(format string, args ...any) { t.Logf(format, args...) })
@ -176,8 +178,8 @@ func TestDERP(t *testing.T) {
assert.True(t, node.CanExchangeMessages)
assert.NotEmpty(t, node.RoundTripPing)
assert.Len(t, node.ClientLogs, 2)
assert.Len(t, node.ClientLogs[0], 3)
assert.Len(t, node.ClientLogs[1], 3)
assert.Len(t, node.ClientLogs[0], 5)
assert.Len(t, node.ClientLogs[1], 5)
assert.Len(t, node.ClientErrs, 2)
assert.Len(t, node.ClientErrs[0], 1) // this
assert.Len(t, node.ClientErrs[1], 1)