-
Notifications
You must be signed in to change notification settings - Fork 15
Open
Description
This is affecting build stability in the repo:
Tooks like buildTLSListener is being allowed to run on a routine and not being waited to complete before the end of a test.
? github.com/hyperledger/firefly-common/pkg/i18n/es [no test files]
time="2024-03-13T23:29:54Z" level=error msg="Failed TLS DN check: Does not match CN =~ /^right\\.example\\.com$/"
time="2024-03-13T23:29:54Z" level=error msg="Failed TLS DN check: Nil cert chain"
time="2024-03-13T23:29:54Z" level=error msg="Failed TLS DN check: Empty cert chain 0"
PASS
panic: Log in goroutine after TestConnectSkipVerification has completed: read failed: EOF
goroutine 41 [running]:
testing.(*common).logDepth(0xc000244340, {0xc000014470, 0x10}, 0x3)
/opt/hostedtoolcache/go/1.21.8/x64/src/testing/testing.go:1022 +0x4c5
testing.(*common).log(...)
/opt/hostedtoolcache/go/1.21.8/x64/src/testing/testing.go:1004
testing.(*common).Logf(0xc000244340, {0x79eba9?, 0x1?}, {0xc000509fc0?, 0x0?, 0x538440?})
/opt/hostedtoolcache/go/1.21.8/x64/src/testing/testing.go:1055 +0x54
github.com/hyperledger/firefly-common/pkg/fftls.buildTLSListener.func1()
/home/runner/work/firefly-common/firefly-common/pkg/fftls/fftls_test.go:115 +0x5e
created by github.com/hyperledger/firefly-common/pkg/fftls.buildTLSListener in goroutine 40
/home/runner/work/firefly-common/firefly-common/pkg/fftls/fftls_test.go:103 +0x111
Metadata
Metadata
Assignees
Labels
No labels