mirror of
https://github.com/caddyserver/caddy.git
synced 2024-12-16 21:56:40 -05:00
bd17eb205d
* ci: Use golangci's github action for linting Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix most of the staticcheck lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the prealloc lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the misspell lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the varcheck lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the errcheck lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the bodyclose lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the deadcode lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the unused lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the gosec lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the gosimple lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the ineffassign lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Fix the staticcheck lint errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Revert the misspell change, use a neutral English Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Remove broken golangci-lint CI job Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Re-add errantly-removed weakrand initialization Signed-off-by: Dave Henderson <dhenderson@gmail.com> * don't break the loop and return * Removing extra handling for null rootKey * unignore RegisterModule/RegisterAdapter Co-authored-by: Mohammed Al Sahaf <msaa1990@gmail.com> * single-line log message Co-authored-by: Matt Holt <mholt@users.noreply.github.com> * Fix lint after a1808b0dbf209c615e438a496d257ce5e3acdce2 was merged Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Revert ticker change, ignore it instead Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Ignore some of the write errors Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Remove blank line Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Use lifetime Signed-off-by: Dave Henderson <dhenderson@gmail.com> * close immediately Co-authored-by: Matt Holt <mholt@users.noreply.github.com> * Preallocate configVals Signed-off-by: Dave Henderson <dhenderson@gmail.com> * Update modules/caddytls/distributedstek/distributedstek.go Co-authored-by: Mohammed Al Sahaf <msaa1990@gmail.com> Co-authored-by: Matt Holt <mholt@users.noreply.github.com>
98 lines
2.1 KiB
YAML
98 lines
2.1 KiB
YAML
linters-settings:
|
|
errcheck:
|
|
ignore: fmt:.*,io/ioutil:^Read.*,go.uber.org/zap/zapcore:^Add.*
|
|
ignoretests: true
|
|
|
|
linters:
|
|
disable-all: true
|
|
enable:
|
|
- bodyclose
|
|
- deadcode
|
|
- errcheck
|
|
- gofmt
|
|
- goimports
|
|
- gosec
|
|
- gosimple
|
|
- govet
|
|
- ineffassign
|
|
- misspell
|
|
- prealloc
|
|
- staticcheck
|
|
- structcheck
|
|
- typecheck
|
|
- unconvert
|
|
- unused
|
|
- varcheck
|
|
# these are implicitly disabled:
|
|
# - asciicheck
|
|
# - depguard
|
|
# - dogsled
|
|
# - dupl
|
|
# - exhaustive
|
|
# - exportloopref
|
|
# - funlen
|
|
# - gci
|
|
# - gochecknoglobals
|
|
# - gochecknoinits
|
|
# - gocognit
|
|
# - goconst
|
|
# - gocritic
|
|
# - gocyclo
|
|
# - godot
|
|
# - godox
|
|
# - goerr113
|
|
# - gofumpt
|
|
# - goheader
|
|
# - golint
|
|
# - gomnd
|
|
# - gomodguard
|
|
# - goprintffuncname
|
|
# - interfacer
|
|
# - lll
|
|
# - maligned
|
|
# - nakedret
|
|
# - nestif
|
|
# - nlreturn
|
|
# - noctx
|
|
# - nolintlint
|
|
# - rowserrcheck
|
|
# - scopelint
|
|
# - sqlclosecheck
|
|
# - stylecheck
|
|
# - testpackage
|
|
# - unparam
|
|
# - whitespace
|
|
# - wsl
|
|
|
|
run:
|
|
# default concurrency is a available CPU number.
|
|
# concurrency: 4 # explicitly omit this value to fully utilize available resources.
|
|
deadline: 5m
|
|
issues-exit-code: 1
|
|
tests: false
|
|
|
|
# output configuration options
|
|
output:
|
|
format: 'colored-line-number'
|
|
print-issued-lines: true
|
|
print-linter-name: true
|
|
|
|
issues:
|
|
exclude-rules:
|
|
# we aren't calling unknown URL
|
|
- text: "G107" # G107: Url provided to HTTP request as taint input
|
|
linters:
|
|
- gosec
|
|
# as a web server that's expected to handle any template, this is totally in the hands of the user.
|
|
- text: "G203" # G203: Use of unescaped data in HTML templates
|
|
linters:
|
|
- gosec
|
|
# we're shelling out to known commands, not relying on user-defined input.
|
|
- text: "G204" # G204: Audit use of command execution
|
|
linters:
|
|
- gosec
|
|
# the choice of weakrand is deliberate, hence the named import "weakrand"
|
|
- path: modules/caddyhttp/reverseproxy/selectionpolicies.go
|
|
text: "G404" # G404: Insecure random number source (rand)
|
|
linters:
|
|
- gosec
|