Addressing comment on PR #39 #47
CopilotAnnotations
10 errors
|
copilot:
modules/httpserver/module_test.go#L316
require-error: for error assertions use require (testifylint)
|
|
copilot:
modules/httpserver/module_test.go#L224
require-error: for error assertions use require (testifylint)
|
|
copilot:
modules/httpserver/module_test.go#L180
require-error: for error assertions use require (testifylint)
|
|
copilot:
modules/httpserver/module_test.go#L428
(*net/http.Client).Get must not be called. use (*net/http.Client).Do(*http.Request) (noctx)
|
|
copilot:
modules/httpserver/module_test.go#L286
(*net/http.Client).Get must not be called. use (*net/http.Client).Do(*http.Request) (noctx)
|
|
copilot:
modules/httpserver/module_test.go#L423
G402: TLS InsecureSkipVerify set true. (gosec)
|
|
copilot:
modules/httpserver/certificate_service_test.go#L207
G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
|
|
copilot:
modules/httpserver/certificate_service_test.go#L67
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"config section %s not found\", name)" (err113)
|
|
copilot:
modules/httpserver/certificate_service_test.go#L33
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"no certificate found for domain: %s\", clientHello.ServerName)" (err113)
|
|
copilot:
modules/httpserver/certificate_service_test.go#L28
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"server name is empty\")" (err113)
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
results
Expired
|
1.77 KB |
sha256:6f57f4827e435305311ad73d2e7307a948413e475c583258414ec759553675f1
|
|