Skip to content
build

bug: propagate TLS errors as failures #308

Sign in to view logs

bug: propagate TLS errors as failures

bug: propagate TLS errors as failures #308

Triggered via pull request April 15, 2024 14:54
@ainghazalainghazal
synchronize #73
ainghazal:bug/tls-errors
Status Success
Total duration 48s
Artifacts –

build.yml

on: pull_request
short-tests
27s
short-tests
lint
5s
lint
gosec
39s
gosec
coverage-threshold
30s
coverage-threshold
integration
35s
integration
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
lint: internal/session/manager.go#L133
comment on exported method Manager.NewACKForPacketIDs should be of the form "NewACKForPacketIDs ..."
lint: internal/session/manager.go#L166
if block ends with a return statement, so drop this else and outdent its block
lint: internal/runtimex/runtimex_test.go#L13
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/session/manager.go#L267
exported method Manager.CurrentKeyID should have comment or be unexported
lint: internal/runtimex/runtimex_test.go#L31
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/runtimex/runtimex_test.go#L22
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/model/mocks.go#L5
exported type TestLogger should have comment or be unexported
lint: internal/runtimex/runtimex_test.go#L40
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/model/mocks.go#L16
exported method TestLogger.Debugf should have comment or be unexported
lint: internal/model/mocks.go#L13
exported method TestLogger.Debug should have comment or be unexported