*: support TLS/mTLS on the peer and client #1433
Annotations
12 errors and 12 warnings
Maelstrom:
maelstrom/main.go#L191
cannot use grpcProvider (variable of type *maelstromGrpcProvider) as container.GrpcProvider value in argument to server.NewWithGrpcProvider: *maelstromGrpcProvider does not implement container.GrpcProvider (wrong type for method StartGrpcServer)
|
Maelstrom
Process completed with exit code 2.
|
Run chaos-mesh
Canceling since a higher priority waiting request for 'Build-refs/pull/443/merge' exists
|
Run chaos-mesh
The operation was canceled.
|
Build & Test
Canceling since a higher priority waiting request for 'Build-refs/pull/443/merge' exists
|
Build & Test:
maelstrom/coordinator_rpc_client.go#L1
: # github.com/streamnative/oxia/maelstrom
|
Build & Test:
maelstrom/main.go#L191
cannot use grpcProvider (variable of type *maelstromGrpcProvider) as container.GrpcProvider value in argument to server.NewWithGrpcProvider: *maelstromGrpcProvider does not implement container.GrpcProvider (wrong type for method StartGrpcServer)
|
Build & Test:
common/security/tls.go#L78
G402: TLS InsecureSkipVerify may be true. (gosec)
|
Build & Test:
common/security/tls.go#L94
ineffectual assignment to bPem (ineffassign)
|
Build & Test:
tests/security/tls/tls_encryption_test.go#L157
ineffectual assignment to err (ineffassign)
|
Build & Test:
tests/security/tls/tls_encryption_test.go#L196
ineffectual assignment to err (ineffassign)
|
Build & Test
issues found
|
Maelstrom
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v3, actions/setup-java@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Maelstrom
No files were found with the provided path: maelstrom/store. No artifacts will be uploaded.
|
Build & Test:
common/security/tls.go#L25
var-naming: type TlsOption should be TLSOption (revive)
|
Build & Test:
common/security/tls.go#L47
var-naming: var InvalidTlsCertFile should be InvalidTLSCertFile (revive)
|
Build & Test:
common/security/tls.go#L48
var-naming: var InvalidTlsKeyFile should be InvalidTLSKeyFile (revive)
|
Build & Test:
common/security/tls.go#L105
var-naming: method MakeClientTlsConf should be MakeClientTLSConf (revive)
|
Build & Test:
common/security/tls.go#L126
var-naming: method MakeServerTlsConf should be MakeServerTLSConf (revive)
|
Build & Test:
common/container/container.go#L46
import-shadowing: The name 'tls' shadows an import name (revive)
|
Build & Test:
common/container/container.go#L54
import-shadowing: The name 'tls' shadows an import name (revive)
|
Build & Test:
common/container/container.go#L66
import-shadowing: The name 'tls' shadows an import name (revive)
|
Build & Test:
oxia/options.go#L48
var-naming: var ErrInvalidOptionTls should be ErrInvalidOptionTLS (revive)
|
Build & Test:
tests/security/tls/tls_encryption_test.go#L32
var-naming: func getPeerTlsOption should be getPeerTLSOption (revive)
|