fix failing test #64
alexmarnellAnnotations
10 errors and 2 warnings
|
ci (lint):
hmiddleware/httpmetrics/httpmetrics.go#L54
string `http.server.` has 3 occurrences, make it a constant (goconst)
|
|
ci (lint):
cmd/protoc-gen-loggingtags/golden_test.go#L108
badCall: suspicious Join on 1 argument (gocritic)
|
|
ci (lint):
hmetrics/examples_test.go#L25
G114: Use of net/http serve function that has no support for setting timeouts (gosec)
|
|
ci (lint):
hmetrics/examples_test.go#L65
G114: Use of net/http serve function that has no support for setting timeouts (gosec)
|
|
ci (lint):
hmetrics/example/basic/main.go#L26
G114: Use of net/http serve function that has no support for setting timeouts (gosec)
|
|
ci (lint):
grpc/grpcserver/server.go#L101
G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
|
|
ci (lint):
cmdutil/service/standard_http.go#L47
G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
|
|
ci (lint):
cmdutil/service/standard_http.go#L56
G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
|
|
ci (lint):
go-kit/metrics/provider/otel/options.go#L37
const `exporterAlreadyCreatedPanic` is unused (unused)
|
|
ci (lint):
cmd/protoc-gen-loggingtags/internal/gen/generate.go#L164
SA1019: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead. (staticcheck)
|
|
ci (test)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
ci (lint)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|