diff options
| author | Jacob Gadikian <[email protected]> | 2023-08-14 23:41:15 +0800 |
|---|---|---|
| committer | GitHub <[email protected]> | 2023-08-14 09:41:15 -0600 |
| commit | d6f86cccf5fa5b4eb30141da390cf2439746c5da (patch) | |
| tree | cd8ad7aa2175259534b572d3a1b6c4852ef11343 /.golangci.yml | |
| parent | 2d7d806fcf91402eb15ae3ffd916bf0b2c6508df (diff) | |
| download | caddy-d6f86cccf5fa5b4eb30141da390cf2439746c5da.tar.gz caddy-d6f86cccf5fa5b4eb30141da390cf2439746c5da.zip | |
ci: use gci linter (#5708)
* use gofmput to format code
* use gci to format imports
* reconfigure gci
* linter autofixes
* rearrange imports a little
* export GOOS=windows golangci-lint run ./... --fix
Diffstat (limited to '.golangci.yml')
| -rw-r--r-- | .golangci.yml | 24 |
1 files changed, 19 insertions, 5 deletions
diff --git a/.golangci.yml b/.golangci.yml index 1dc305e0b..5f018970e 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -2,12 +2,26 @@ linters-settings: errcheck: ignore: fmt:.*,go.uber.org/zap/zapcore:^Add.* ignoretests: true + gci: + sections: + - standard # Standard section: captures all standard packages. + - default # Default section: contains all imports that could not be matched to another section type. + - prefix(github.com/caddyserver/caddy/v2/cmd) # ensure that this is always at the top and always has a line break. + - prefix(github.com/caddyserver/caddy) # Custom section: groups all imports with the specified Prefix. + # Skip generated files. + # Default: true + skip-generated: true + # Enable custom order of sections. + # If `true`, make the section order the same as the order of `sections`. + # Default: false + custom-order: true linters: disable-all: true enable: - bodyclose - errcheck + - gci - gofumpt - gosec - gosimple @@ -76,23 +90,23 @@ output: issues: exclude-rules: # we aren't calling unknown URL - - text: "G107" # G107: Url provided to HTTP request as taint input + - 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 + - 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 + - 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) + text: 'G404' # G404: Insecure random number source (rand) linters: - gosec - path: modules/caddyhttp/reverseproxy/streaming.go - text: "G404" # G404: Insecure random number source (rand) + text: 'G404' # G404: Insecure random number source (rand) linters: - gosec |