Refactor server and proxy and remove gnet/v2
#814
mostafatest.yaml
on: pull_request
Test GatewayD
1m 15s
Test GatewayD Plugins
0s
Annotations
10 errors and 3 warnings
|
Test GatewayD:
network/engine.go#L120
Error return value of `conn.Write` is not checked (errcheck)
|
|
Test GatewayD:
config/getters.go#L31
var `loadBalancers` is unused (unused)
|
|
Test GatewayD:
network/engine.go#L42
mnd: Magic number: 2, in <argument> detected (gomnd)
|
|
Test GatewayD:
network/engine.go#L85
S1000: should use for range instead of for { select {} } (gosimple)
|
|
Test GatewayD:
network/engine.go#L139
S1000: should use for range instead of for { select {} } (gosimple)
|
|
Test GatewayD:
network/proxy.go#L671
variable name 'n' is too short for the scope of its usage (varnamelen)
|
|
Test GatewayD:
network/client.go#L156
variable name 'n' is too short for the scope of its usage (varnamelen)
|
|
Test GatewayD:
network/engine.go#L62
error returned from external package is unwrapped: sig: func net.Listen(network string, address string) (net.Listener, error) (wrapcheck)
|
|
Test GatewayD:
network/engine.go#L75
error returned from external package is unwrapped: sig: func net.SplitHostPort(hostport string) (host string, port string, err error) (wrapcheck)
|
|
Test GatewayD:
network/engine.go#L80
error returned from external package is unwrapped: sig: func strconv.Atoi(s string) (int, error) (wrapcheck)
|
|
Test GatewayD:
network/engine.go#L63
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
|
Test GatewayD:
network/engine.go#L76
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
|
Test GatewayD:
network/engine.go#L50
unused-parameter: parameter 'opts' seems to be unused, consider removing or renaming it as _ (revive)
|