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/derp/derphttp
Alex Chan 7dfa26778e derp/derphttp: de-flake DERP HTTP clients tests with memnet and synctest
Using memnet and synctest removes flakiness caused by real networking
and subtle timing differences.

Additionally, remove the `t.Logf` call inside the server's shutdown
goroutine that was causing a false positive data race detection.

The race detector is flagging a double write during this `t.Logf` call.
This is a common pattern, noted in golang/go#40343 and elsehwere in
this file, where using `t.Logf` after a test has finished can interact
poorly with the test runner.

This is a long-standing issue which became more common after rewriting
this test to use memnet and synctest.

Fixed #17355

Signed-off-by: Alex Chan <alexc@tailscale.com>
2 months ago
..
derphttp_client.go feature, net/tshttpproxy: pull out support for using proxies as a feature 2 months ago
derphttp_test.go derp/derphttp: de-flake DERP HTTP clients tests with memnet and synctest 2 months ago
export_test.go derp/derpserver: split off derp.Server out of derp into its own package 2 months ago
mesh_client.go derp/derphttp: add error notify for RunWatchConnectionLoop (#16261) 6 months ago
websocket.go derp/derphttp: don't link websockets other than on GOOS=js 1 year ago
websocket_stub.go derp/derphttp: don't link websockets other than on GOOS=js 1 year ago