getlantern / tlsmasq

A library for servers which masquerade as other TLS servers
Apache License 2.0
2 stars 1 forks source link

Resolve new tlsmasq test failures in Go 1.16 #22

Closed hwh33 closed 3 years ago

hwh33 commented 3 years ago

Switching to Go 1.16 seems to have exposed new test failures. Some of these are likely due to improvements in the race detector (https://golang.org/doc/go1.16#runtime). However, some of the new failures occur even when the tests are run without the race detector. The goal should be running the tests successfully with -race -count=1000.

This is possibly related to #14 and #21.

hwh33 commented 3 years ago

This seems to have fixed most of the bugs, including the particularly strange ones: https://github.com/getlantern/tlsmasq/pull/24

The tests occasionally still hang, so this is not totally resolved.