AdguardTeam / AdguardTeam/dnsproxy

*testing.T.Fatalf() is called in sendTestMessageAsync() from mutiple goroutines

Đang mở
#112 0 bình luận 1 reaction 1 người được giao Được @ainar-g nhận Xem trên GitHub
good first issue
Ngôn ngữ chính
Go
Star
3.3k
Fork
343
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

https://github.com/AdguardTeam/dnsproxy/blob/2f7a50c830677fd6fed48a071a5f8f4ff563c1a8/proxy/proxy_test.go#L751

https://github.com/AdguardTeam/dnsproxy/blob/2f7a50c830677fd6fed48a071a5f8f4ff563c1a8/proxy/proxy_test.go#L734

The golang comment says that `Fatalf()` `must be called only from the goroutine running the Test function.`

https://github.com/golang/go/blob/926994fd7cf65b2703552686965fb05569699897/src/testing/testing.go#L659

```
// T is a type passed to Test functions to manage test state and support formatted test logs.
//
// A test ends when its Test function returns or calls any of the methods
// FailNow, Fatal, Fatalf, SkipNow, Skip, or Skipf. Those methods, as well as
// the Parallel method, must be called only from the goroutine running the
// Test function.
```

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.