Merge challtestsrv package into Pebble #107
mcpherrinmchecks.yml
on: pull_request
go-lint-checks
24s
go-mod-checks
11s
Annotations
9 errors
|
go-lint-checks:
challtestsrv/tlsalpnone.go#L72
`marshalling` is a misspelling of `marshaling` (misspell)
|
|
go-lint-checks:
challtestsrv/dnsone.go#L75
dohServer - result 1 (error) is always nil (unparam)
|
|
go-lint-checks:
challtestsrv/tlsalpnone.go#L87
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
|
|
go-lint-checks:
challtestsrv/event.go#L73
S1017: should replace this if statement with an unconditional strings.TrimSuffix (gosimple)
|
|
go-lint-checks:
challtestsrv/dns.go#L161
interface method WriteMsg must have all named params (inamedparam)
|
|
go-lint-checks:
challtestsrv/httpone.go#L184
httpOneServer returns interface (github.com/letsencrypt/pebble/v2/challtestsrv.challengeServer) (ireturn)
|
|
go-lint-checks:
challtestsrv/tlsalpnone.go#L110
tlsALPNOneServer returns interface (github.com/letsencrypt/pebble/v2/challtestsrv.challengeServer) (ireturn)
|
|
go-lint-checks:
challtestsrv/challenge-servers.go#L121
fmt.Errorf can be replaced with errors.New (perfsprint)
|
|
go-lint-checks
issues found
|