diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 43208cac..fc901591 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -1,7 +1,7 @@ 'name': 'build' 'env': - 'GO_VERSION': '1.23.2' + 'GO_VERSION': '1.23.3' 'NODE_VERSION': '16' 'on': diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 526c70fe..3d24f103 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -1,7 +1,7 @@ 'name': 'lint' 'env': - 'GO_VERSION': '1.23.2' + 'GO_VERSION': '1.23.3' 'on': 'push': diff --git a/CHANGELOG.md b/CHANGELOG.md index 0714cf29..f248d42c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -27,8 +27,13 @@ NOTE: Add new changes BELOW THIS COMMENT. ### Security +- Go version has been updated to prevent the possibility of exploiting the Go + vulnerabilities fixed in [1.23.3][go-1.23.3]. + - The release executables are now signed. +[go-1.23.3]: https://groups.google.com/g/golang-announce/c/X5KodEJYuqI + diff --git a/Makefile b/Makefile index 2d891860..7849d724 100644 --- a/Makefile +++ b/Makefile @@ -27,8 +27,8 @@ DEPLOY_SCRIPT_PATH = not/a/real/path DIST_DIR = dist GOAMD64 = v1 GOPROXY = https://proxy.golang.org|direct -GOTOOLCHAIN = go1.23.2 GOTELEMETRY = off +GOTOOLCHAIN = go1.23.3 GPG_KEY = devteam@adguard.com GPG_KEY_PASSPHRASE = not-a-real-password NPM = npm diff --git a/bamboo-specs/release.yaml b/bamboo-specs/release.yaml index 182f327d..889e53fa 100644 --- a/bamboo-specs/release.yaml +++ b/bamboo-specs/release.yaml @@ -8,7 +8,7 @@ 'variables': 'channel': 'edge' 'dockerFrontend': 'adguard/home-js-builder:2.0' - 'dockerGo': 'adguard/go-builder:1.23.2--1' + 'dockerGo': 'adguard/go-builder:1.23.3--1' 'stages': - 'Build frontend': @@ -276,7 +276,7 @@ 'variables': 'channel': 'beta' 'dockerFrontend': 'adguard/home-js-builder:2.0' - 'dockerGo': 'adguard/go-builder:1.23.2--1' + 'dockerGo': 'adguard/go-builder:1.23.3--1' # release-vX.Y.Z branches are the branches from which the actual final # release is built. - '^release-v[0-9]+\.[0-9]+\.[0-9]+': @@ -292,4 +292,4 @@ 'variables': 'channel': 'release' 'dockerFrontend': 'adguard/home-js-builder:2.0' - 'dockerGo': 'adguard/go-builder:1.23.2--1' + 'dockerGo': 'adguard/go-builder:1.23.3--1' diff --git a/bamboo-specs/test.yaml b/bamboo-specs/test.yaml index 2f672e2d..c02d208b 100644 --- a/bamboo-specs/test.yaml +++ b/bamboo-specs/test.yaml @@ -6,7 +6,7 @@ 'name': 'AdGuard Home - Build and run tests' 'variables': 'dockerFrontend': 'adguard/home-js-builder:2.0' - 'dockerGo': 'adguard/go-builder:1.23.2--1' + 'dockerGo': 'adguard/go-builder:1.23.3--1' 'channel': 'development' 'stages': @@ -196,5 +196,5 @@ # may need to build a few of these. 'variables': 'dockerFrontend': 'adguard/home-js-builder:2.0' - 'dockerGo': 'adguard/go-builder:1.23.2--1' + 'dockerGo': 'adguard/go-builder:1.23.3--1' 'channel': 'candidate' diff --git a/go.mod b/go.mod index 89556346..2ac94965 100644 --- a/go.mod +++ b/go.mod @@ -1,11 +1,11 @@ module github.com/AdguardTeam/AdGuardHome -go 1.23.2 +go 1.23.3 require ( // TODO(a.garipov): Update when v0.73.3 is released. github.com/AdguardTeam/dnsproxy v0.73.3-0.20241004151328-c7c7b977a2a3 - github.com/AdguardTeam/golibs v0.30.1 + github.com/AdguardTeam/golibs v0.30.2 github.com/AdguardTeam/urlfilter v0.20.0 github.com/NYTimes/gziphandler v1.1.1 github.com/ameshkov/dnscrypt/v2 v2.3.0 @@ -13,7 +13,7 @@ require ( github.com/c2h5oh/datasize v0.0.0-20231215233829-aa82cc1e6500 github.com/digineo/go-ipset/v2 v2.2.1 github.com/dimfeld/httptreemux/v5 v5.5.0 - github.com/fsnotify/fsnotify v1.7.0 + github.com/fsnotify/fsnotify v1.8.0 github.com/go-ping/ping v1.1.0 github.com/google/go-cmp v0.6.0 github.com/google/gopacket v1.1.19 @@ -49,9 +49,9 @@ require ( github.com/beefsack/go-rate v0.0.0-20220214233405-116f4ca011a0 // indirect github.com/davecgh/go-spew v1.1.1 // indirect github.com/go-task/slim-sprig/v3 v3.0.0 // indirect - github.com/google/pprof v0.0.0-20241029010322-833c56d90c8e // indirect + github.com/google/pprof v0.0.0-20241101162523-b92577c0c142 // indirect github.com/mdlayher/socket v0.5.1 // indirect - github.com/onsi/ginkgo/v2 v2.20.2 // indirect + github.com/onsi/ginkgo/v2 v2.21.0 // indirect github.com/patrickmn/go-cache v2.1.0+incompatible // indirect github.com/pierrec/lz4/v4 v4.1.21 // indirect github.com/pkg/errors v0.9.1 // indirect diff --git a/go.sum b/go.sum index 492565f1..a5f8a10f 100644 --- a/go.sum +++ b/go.sum @@ -1,7 +1,7 @@ github.com/AdguardTeam/dnsproxy v0.73.3-0.20241004151328-c7c7b977a2a3 h1:IGXwBjdKDzUm007QzZyxSllMnkbdXe7K79x7JWcBW/E= github.com/AdguardTeam/dnsproxy v0.73.3-0.20241004151328-c7c7b977a2a3/go.mod h1:356iHROxo+SOdBVifp1MXEh6qHyydtzGCcsQMfx+ZVs= -github.com/AdguardTeam/golibs v0.30.1 h1:/yv7dq2h7WXw/jTDxkE3FP9zHerRT+i03PZRHJX4fPU= -github.com/AdguardTeam/golibs v0.30.1/go.mod h1:FkwcNQEJoGsgDGXcalrVa/4gWbE68KsmE2guXWtBQUE= +github.com/AdguardTeam/golibs v0.30.2 h1:urU/NAyIvQOeArBqDmKCDpaRkfTCJ26uSiSuDMKQfuY= +github.com/AdguardTeam/golibs v0.30.2/go.mod h1:FkwcNQEJoGsgDGXcalrVa/4gWbE68KsmE2guXWtBQUE= github.com/AdguardTeam/urlfilter v0.20.0 h1:X32qiuVCVd8WDYCEsbdZKfXMzwdVqrdulamtUi4rmzs= github.com/AdguardTeam/urlfilter v0.20.0/go.mod h1:gjrywLTxfJh6JOkwi9SU+frhP7kVVEZ5exFGkR99qpk= github.com/NYTimes/gziphandler v1.1.1 h1:ZUDjpQae29j0ryrS0u/B8HZfJBtBQHjqw2rQ2cqUQ3I= @@ -27,8 +27,8 @@ github.com/digineo/go-ipset/v2 v2.2.1 h1:k6skY+0fMqeUjjeWO/m5OuWPSZUAn7AucHMnQ1M github.com/digineo/go-ipset/v2 v2.2.1/go.mod h1:wBsNzJlZlABHUITkesrggFnZQtgW5wkqw1uo8Qxe0VU= github.com/dimfeld/httptreemux/v5 v5.5.0 h1:p8jkiMrCuZ0CmhwYLcbNbl7DDo21fozhKHQ2PccwOFQ= github.com/dimfeld/httptreemux/v5 v5.5.0/go.mod h1:QeEylH57C0v3VO0tkKraVz9oD3Uu93CKPnTLbsidvSw= -github.com/fsnotify/fsnotify v1.7.0 h1:8JEhPFa5W2WU7YfeZzPNqzMP6Lwt7L2715Ggo0nosvA= -github.com/fsnotify/fsnotify v1.7.0/go.mod h1:40Bi/Hjc2AVfZrqy+aj+yEI+/bRxZnMJyTJwOpGvigM= +github.com/fsnotify/fsnotify v1.8.0 h1:dAwr6QBTBZIkG8roQaJjGof0pp0EeF+tNV7YBP3F/8M= +github.com/fsnotify/fsnotify v1.8.0/go.mod h1:8jBTzvmWwFyi3Pb8djgCCO5IBqzKJ/Jwo8TRcHyHii0= github.com/go-logr/logr v1.4.2 h1:6pFjapn8bFcIbiKo3XT4j/BhANplGihG6tvd+8rYgrY= github.com/go-logr/logr v1.4.2/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY= github.com/go-ole/go-ole v1.2.6 h1:/Fpf6oFPoeFik9ty7siob0G6Ke8QvQEuVcuChpwXzpY= @@ -44,8 +44,8 @@ github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI= github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY= github.com/google/gopacket v1.1.19 h1:ves8RnFZPGiFnTS0uPQStjwru6uO6h+nlr9j6fL7kF8= github.com/google/gopacket v1.1.19/go.mod h1:iJ8V8n6KS+z2U1A8pUwu8bW5SyEMkXJB8Yo/Vo+TKTo= -github.com/google/pprof v0.0.0-20241029010322-833c56d90c8e h1:v7R0PZoC2p1KWQmv1+GqCXQe59Ab1TkDF8Y9Lg2W6m4= -github.com/google/pprof v0.0.0-20241029010322-833c56d90c8e/go.mod h1:vavhavw2zAxS5dIdcRluK6cSGGPlZynqzFM8NdvU144= +github.com/google/pprof v0.0.0-20241101162523-b92577c0c142 h1:sAGdeJj0bnMgUNVeUpp6AYlVdCt3/GdI3pGRqsNSQLs= +github.com/google/pprof v0.0.0-20241101162523-b92577c0c142/go.mod h1:vavhavw2zAxS5dIdcRluK6cSGGPlZynqzFM8NdvU144= github.com/google/renameio/v2 v2.0.0 h1:UifI23ZTGY8Tt29JbYFiuyIU3eX+RNFtUwefq9qAhxg= github.com/google/renameio/v2 v2.0.0/go.mod h1:BtmJXm5YlszgC+TD4HOEEUFgkJP3nLxehU6hfe7jRt4= github.com/google/uuid v1.2.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo= @@ -82,10 +82,10 @@ github.com/miekg/dns v1.1.62 h1:cN8OuEF1/x5Rq6Np+h1epln8OiyPWV+lROx9LxcGgIQ= github.com/miekg/dns v1.1.62/go.mod h1:mvDlcItzm+br7MToIKqkglaGhlFMHJ9DTNNWONWXbNQ= github.com/niemeyer/pretty v0.0.0-20200227124842-a10e7caefd8e h1:fD57ERR4JtEqsWbfPhv4DMiApHyliiK5xCTNVSPiaAs= github.com/niemeyer/pretty v0.0.0-20200227124842-a10e7caefd8e/go.mod h1:zD1mROLANZcx1PVRCS0qkT7pwLkGfwJo4zjcN/Tysno= -github.com/onsi/ginkgo/v2 v2.20.2 h1:7NVCeyIWROIAheY21RLS+3j2bb52W0W82tkberYytp4= -github.com/onsi/ginkgo/v2 v2.20.2/go.mod h1:K9gyxPIlb+aIvnZ8bd9Ak+YP18w3APlR+5coaZoE2ag= -github.com/onsi/gomega v1.34.1 h1:EUMJIKUjM8sKjYbtxQI9A4z2o+rruxnzNvpknOXie6k= -github.com/onsi/gomega v1.34.1/go.mod h1:kU1QgUvBDLXBJq618Xvm2LUX6rSAfRaFRTcdOeDLwwY= +github.com/onsi/ginkgo/v2 v2.21.0 h1:7rg/4f3rB88pb5obDgNZrNHrQ4e6WpjonchcpuBRnZM= +github.com/onsi/ginkgo/v2 v2.21.0/go.mod h1:7Du3c42kxCUegi0IImZ1wUQzMBVecgIHjR1C+NkhLQo= +github.com/onsi/gomega v1.34.2 h1:pNCwDkzrsv7MS9kpaQvVb1aVLahQXyJ/Tv5oAZMI3i8= +github.com/onsi/gomega v1.34.2/go.mod h1:v1xfxRgk0KIsG+QOdm7p8UosrOzPYRo60fd3B/1Dukc= github.com/patrickmn/go-cache v2.1.0+incompatible h1:HRMgzkcYKYpi3C8ajMPV8OFXaaRUnok+kx1WdO15EQc= github.com/patrickmn/go-cache v2.1.0+incompatible/go.mod h1:3Qf8kWWT7OJRJbdiICTKqZju1ZixQ/KpMGzzAfe6+WQ= github.com/pierrec/lz4/v4 v4.1.21 h1:yOVMLb6qSIDP67pl/5F7RepeKYu/VmTyEXvuMI5d9mQ= diff --git a/internal/client/index.go b/internal/client/index.go index cad6deb3..2eb7411b 100644 --- a/internal/client/index.go +++ b/internal/client/index.go @@ -2,6 +2,7 @@ package client import ( "fmt" + "maps" "net" "net/netip" "slices" @@ -9,7 +10,6 @@ import ( "github.com/AdguardTeam/AdGuardHome/internal/aghalg" "github.com/AdguardTeam/golibs/errors" - "golang.org/x/exp/maps" ) // macKey contains MAC as byte array of 6, 8, or 20 bytes. @@ -330,12 +330,14 @@ func (ci *index) size() (n int) { // rangeByName is like [Index.Range] but sorts the persistent clients by name // before iterating ensuring a predictable order. func (ci *index) rangeByName(f func(c *Persistent) (cont bool)) { - cs := maps.Values(ci.uidToClient) - slices.SortFunc(cs, func(a, b *Persistent) (n int) { - return strings.Compare(a.Name, b.Name) - }) + clients := slices.SortedStableFunc( + maps.Values(ci.uidToClient), + func(a, b *Persistent) (res int) { + return strings.Compare(a.Name, b.Name) + }, + ) - for _, c := range cs { + for _, c := range clients { if !f(c) { break } diff --git a/internal/stats/unit.go b/internal/stats/unit.go index e9aeb87b..cef7fca2 100644 --- a/internal/stats/unit.go +++ b/internal/stats/unit.go @@ -5,6 +5,7 @@ import ( "encoding/binary" "encoding/gob" "fmt" + "maps" "slices" "time" @@ -12,7 +13,6 @@ import ( "github.com/AdguardTeam/golibs/errors" "github.com/AdguardTeam/golibs/logutil/slogutil" "go.etcd.io/bbolt" - "golang.org/x/exp/maps" ) const ( @@ -234,18 +234,15 @@ func (a countPair) compareCount(b countPair) (res int) { } } -func convertMapToSlice(m map[string]uint64, max int) (s []countPair) { +func convertMapToSlice(m map[string]uint64, maxVal int) (s []countPair) { s = make([]countPair, 0, len(m)) for k, v := range m { s = append(s, countPair{Name: k, Count: v}) } slices.SortFunc(s, countPair.compareCount) - if max > len(s) { - max = len(s) - } - return s[:max] + return s[:min(maxVal, len(s))] } func convertSliceToMap(a []countPair) (m map[string]uint64) { @@ -611,9 +608,7 @@ func microsecondsToSeconds(n float64) (r float64) { func prepareTopUpstreamsAvgTime( upstreamsAvgTime topAddrsFloat, ) (topUpstreamsAvgTime []topAddrsFloat) { - keys := maps.Keys(upstreamsAvgTime) - - slices.SortFunc(keys, func(a, b string) (res int) { + keys := slices.SortedStableFunc(maps.Keys(upstreamsAvgTime), func(a, b string) (res int) { switch x, y := upstreamsAvgTime[a], upstreamsAvgTime[b]; { case x > y: return -1 diff --git a/internal/tools/go.mod b/internal/tools/go.mod index ca44f782..2b1b741b 100644 --- a/internal/tools/go.mod +++ b/internal/tools/go.mod @@ -1,27 +1,28 @@ module github.com/AdguardTeam/AdGuardHome/internal/tools -go 1.23.2 +go 1.23.3 require ( github.com/fzipp/gocyclo v0.6.0 github.com/golangci/misspell v0.6.0 github.com/gordonklaus/ineffassign v0.1.0 + github.com/jstemmer/go-junit-report/v2 v2.1.0 github.com/kisielk/errcheck v1.8.0 - github.com/kyoh86/looppointer v0.2.1 github.com/securego/gosec/v2 v2.21.4 github.com/uudashr/gocognit v1.1.3 golang.org/x/tools v0.26.0 golang.org/x/vuln v1.1.3 honnef.co/go/tools v0.5.1 mvdan.cc/gofumpt v0.7.0 + mvdan.cc/sh/v3 v3.10.0 mvdan.cc/unparam v0.0.0-20240917084806-57a3b4290ba3 ) require ( cloud.google.com/go v0.116.0 // indirect cloud.google.com/go/ai v0.8.2 // indirect - cloud.google.com/go/auth v0.9.9 // indirect - cloud.google.com/go/auth/oauth2adapt v0.2.4 // indirect + cloud.google.com/go/auth v0.10.1 // indirect + cloud.google.com/go/auth/oauth2adapt v0.2.5 // indirect cloud.google.com/go/compute/metadata v0.5.2 // indirect cloud.google.com/go/longrunning v0.6.2 // indirect github.com/BurntSushi/toml v1.4.1-0.20240526193622-a339e1f7089c // indirect @@ -32,12 +33,13 @@ require ( github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect github.com/google/generative-ai-go v0.18.0 // indirect github.com/google/go-cmp v0.6.0 // indirect + github.com/google/renameio/v2 v2.0.0 // indirect github.com/google/s2a-go v0.1.8 // indirect github.com/google/uuid v1.6.0 // indirect github.com/googleapis/enterprise-certificate-proxy v0.3.4 // indirect github.com/googleapis/gax-go/v2 v2.13.0 // indirect github.com/gookit/color v1.5.4 // indirect - github.com/kyoh86/nolint v0.0.1 // indirect + github.com/rogpeppe/go-internal v1.13.1 // indirect github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e // indirect go.opencensus.io v0.24.0 // indirect go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.56.0 // indirect @@ -53,13 +55,15 @@ require ( golang.org/x/oauth2 v0.23.0 // indirect golang.org/x/sync v0.8.0 // indirect golang.org/x/sys v0.26.0 // indirect - golang.org/x/telemetry v0.0.0-20241028140143-9c0d19e65ba0 // indirect + golang.org/x/telemetry v0.0.0-20241106142447-58a1122356f5 // indirect + golang.org/x/term v0.25.0 // indirect golang.org/x/text v0.19.0 // indirect golang.org/x/time v0.7.0 // indirect - google.golang.org/api v0.203.0 // indirect - google.golang.org/genproto/googleapis/api v0.0.0-20241021214115-324edc3d5d38 // indirect - google.golang.org/genproto/googleapis/rpc v0.0.0-20241021214115-324edc3d5d38 // indirect + google.golang.org/api v0.205.0 // indirect + google.golang.org/genproto/googleapis/api v0.0.0-20241104194629-dd2ea8efbc28 // indirect + google.golang.org/genproto/googleapis/rpc v0.0.0-20241104194629-dd2ea8efbc28 // indirect google.golang.org/grpc v1.67.1 // indirect google.golang.org/protobuf v1.35.1 // indirect gopkg.in/yaml.v3 v3.0.1 // indirect + mvdan.cc/editorconfig v0.3.0 // indirect ) diff --git a/internal/tools/go.sum b/internal/tools/go.sum index ee408c8a..c7f6bdc0 100644 --- a/internal/tools/go.sum +++ b/internal/tools/go.sum @@ -3,10 +3,10 @@ cloud.google.com/go v0.116.0 h1:B3fRrSDkLRt5qSHWe40ERJvhvnQwdZiHu0bJOpldweE= cloud.google.com/go v0.116.0/go.mod h1:cEPSRWPzZEswwdr9BxE6ChEn01dWlTaF05LiC2Xs70U= cloud.google.com/go/ai v0.8.2 h1:LEaQwqBv+k2ybrcdTtCTc9OPZXoEdcQaGrfvDYS6Bnk= cloud.google.com/go/ai v0.8.2/go.mod h1:Wb3EUUGWwB6yHBaUf/+oxUq/6XbCaU1yh0GrwUS8lr4= -cloud.google.com/go/auth v0.9.9 h1:BmtbpNQozo8ZwW2t7QJjnrQtdganSdmqeIBxHxNkEZQ= -cloud.google.com/go/auth v0.9.9/go.mod h1:xxA5AqpDrvS+Gkmo9RqrGGRh6WSNKKOXhY3zNOr38tI= -cloud.google.com/go/auth/oauth2adapt v0.2.4 h1:0GWE/FUsXhf6C+jAkWgYm7X9tK8cuEIfy19DBn6B6bY= -cloud.google.com/go/auth/oauth2adapt v0.2.4/go.mod h1:jC/jOpwFP6JBxhB3P5Rr0a9HLMC/Pe3eaL4NmdvqPtc= +cloud.google.com/go/auth v0.10.1 h1:TnK46qldSfHWt2a0b/hciaiVJsmDXWy9FqyUan0uYiI= +cloud.google.com/go/auth v0.10.1/go.mod h1:xxA5AqpDrvS+Gkmo9RqrGGRh6WSNKKOXhY3zNOr38tI= +cloud.google.com/go/auth/oauth2adapt v0.2.5 h1:2p29+dePqsCHPP1bqDJcKj4qxRyYCcbzKpFyKGt3MTk= +cloud.google.com/go/auth/oauth2adapt v0.2.5/go.mod h1:AlmsELtlEBnaNTL7jCj8VQFLy6mbZv0s4Q7NGBeQ5E8= cloud.google.com/go/compute/metadata v0.5.2 h1:UxK4uu/Tn+I3p2dYWTfiX4wva7aYlKixAHn3fyqngqo= cloud.google.com/go/compute/metadata v0.5.2/go.mod h1:C66sj2AluDcIqakBq/M8lw8/ybHgOZqin2obFxa/E5k= cloud.google.com/go/longrunning v0.6.2 h1:xjDfh1pQcWPEvnfjZmwjKQEcHnpz6lHjfy7Fo0MK+hc= @@ -67,12 +67,15 @@ github.com/google/go-cmp v0.3.1/go.mod h1:8QqcDgzrUqlUb/G2PQTWiueGozuR1884gddMyw github.com/google/go-cmp v0.4.0/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE= github.com/google/go-cmp v0.5.0/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE= github.com/google/go-cmp v0.5.3/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE= +github.com/google/go-cmp v0.5.8/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY= github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI= github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY= github.com/google/pprof v0.0.0-20240827171923-fa2c70bbbfe5 h1:5iH8iuqE5apketRbSFBy+X1V0o+l+8NF1avt4HWl7cA= github.com/google/pprof v0.0.0-20240827171923-fa2c70bbbfe5/go.mod h1:vavhavw2zAxS5dIdcRluK6cSGGPlZynqzFM8NdvU144= github.com/google/renameio v0.1.0 h1:GOZbcHa3HfsPKPlmyPyN2KEohoMXOhdMbHrvbpl2QaA= github.com/google/renameio v0.1.0/go.mod h1:KWCgfxg9yswjAJkECMjeO8J8rahYeXnNhOm40UhjYkI= +github.com/google/renameio/v2 v2.0.0 h1:UifI23ZTGY8Tt29JbYFiuyIU3eX+RNFtUwefq9qAhxg= +github.com/google/renameio/v2 v2.0.0/go.mod h1:BtmJXm5YlszgC+TD4HOEEUFgkJP3nLxehU6hfe7jRt4= github.com/google/s2a-go v0.1.8 h1:zZDs9gcbt9ZPLV0ndSyQk6Kacx2g/X+SKYovpnz3SMM= github.com/google/s2a-go v0.1.8/go.mod h1:6iNWHTpQ+nfNRN5E00MSdfDwVesa8hhS32PhPO8deJA= github.com/google/uuid v1.1.2/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo= @@ -86,16 +89,14 @@ github.com/gookit/color v1.5.4 h1:FZmqs7XOyGgCAxmWyPslpiok1k05wmY3SJTytgvYFs0= github.com/gookit/color v1.5.4/go.mod h1:pZJOeOS8DM43rXbp4AZo1n9zCU2qjpcRko0b6/QJi9w= github.com/gordonklaus/ineffassign v0.1.0 h1:y2Gd/9I7MdY1oEIt+n+rowjBNDcLQq3RsH5hwJd0f9s= github.com/gordonklaus/ineffassign v0.1.0/go.mod h1:Qcp2HIAYhR7mNUVSIxZww3Guk4it82ghYcEXIAk+QT0= +github.com/jstemmer/go-junit-report/v2 v2.1.0 h1:X3+hPYlSczH9IMIpSC9CQSZA0L+BipYafciZUWHEmsc= +github.com/jstemmer/go-junit-report/v2 v2.1.0/go.mod h1:mgHVr7VUo5Tn8OLVr1cKnLuEy0M92wdRntM99h7RkgQ= github.com/kisielk/errcheck v1.8.0 h1:ZX/URYa7ilESY19ik/vBmCn6zdGQLxACwjAcWbHlYlg= github.com/kisielk/errcheck v1.8.0/go.mod h1:1kLL+jV4e+CFfueBmI1dSK2ADDyQnlrnrY/FqKluHJQ= github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE= github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk= github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY= github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE= -github.com/kyoh86/looppointer v0.2.1 h1:Jx9fnkBj/JrIryBLMTYNTj9rvc2SrPS98Dg0w7fxdJg= -github.com/kyoh86/looppointer v0.2.1/go.mod h1:q358WcM8cMWU+5vzqukvaZtnJi1kw/MpRHQm3xvTrjw= -github.com/kyoh86/nolint v0.0.1 h1:GjNxDEkVn2wAxKHtP7iNTrRxytRZ1wXxLV5j4XzGfRU= -github.com/kyoh86/nolint v0.0.1/go.mod h1:1ZiZZ7qqrZ9dZegU96phwVcdQOMKIqRzFJL3ewq9gtI= github.com/onsi/ginkgo/v2 v2.20.2 h1:7NVCeyIWROIAheY21RLS+3j2bb52W0W82tkberYytp4= github.com/onsi/ginkgo/v2 v2.20.2/go.mod h1:K9gyxPIlb+aIvnZ8bd9Ak+YP18w3APlR+5coaZoE2ag= github.com/onsi/gomega v1.34.2 h1:pNCwDkzrsv7MS9kpaQvVb1aVLahQXyJ/Tv5oAZMI3i8= @@ -103,8 +104,8 @@ github.com/onsi/gomega v1.34.2/go.mod h1:v1xfxRgk0KIsG+QOdm7p8UosrOzPYRo60fd3B/1 github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM= github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4= github.com/prometheus/client_model v0.0.0-20190812154241-14fe0d1b01d4/go.mod h1:xMI15A0UPsDsEKsMN9yxemIoYk6Tm2C1GtYGdfGttqA= -github.com/rogpeppe/go-internal v1.12.0 h1:exVL4IDcn6na9z1rAb56Vxr+CgyK3nn3O+epU5NdKM8= -github.com/rogpeppe/go-internal v1.12.0/go.mod h1:E+RYuTGaKKdloAfM02xzb0FW3Paa99yedzYV+kq4uf4= +github.com/rogpeppe/go-internal v1.13.1 h1:KvO1DLK/DRN07sQ1LQKScxyZJuNnedQ5/wKSR38lUII= +github.com/rogpeppe/go-internal v1.13.1/go.mod h1:uMEvuHeurkdAXX61udpOXGD/AzZDWNMNyH2VO9fmH0o= github.com/securego/gosec/v2 v2.21.4 h1:Le8MSj0PDmOnHJgUATjD96PaXRvCpKC+DGJvwyy0Mlk= github.com/securego/gosec/v2 v2.21.4/go.mod h1:Jtb/MwRQfRxCXyCm1rfM1BEiiiTfUOdyzzAhlr6lUTA= github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME= @@ -121,7 +122,6 @@ github.com/uudashr/gocognit v1.1.3 h1:l+a111VcDbKfynh+airAy/DJQKaXh2m9vkoysMPSZy github.com/uudashr/gocognit v1.1.3/go.mod h1:aKH8/e8xbTRBwjbCkwZ8qt4l2EpKXl31KMHgSS+lZ2U= github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e h1:JVG44RsyaB9T2KIHavMF/ppJZNG9ZpyihvCd0w101no= github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e/go.mod h1:RbqR21r5mrJuqunuUZ/Dhy/avygyECGrLceyNeo4LiM= -github.com/yuin/goldmark v1.2.1/go.mod h1:3hX8gzYuyVAZsxl0MRgGTJEmQBFcNTphYh9decYSb74= github.com/yuin/goldmark v1.3.5/go.mod h1:mwnBkeHKe2W/ZEtQ+71ViKU8L12m81fl3OWwC1Zlc8k= go.opencensus.io v0.24.0 h1:y73uSU6J157QMP2kn2r30vwW1A2W2WFwSCGnAVxeaD0= go.opencensus.io v0.24.0/go.mod h1:vNK8G9p7aAivkbmorf4v+7Hgx+Zs0yY+0fOtgBfjQKo= @@ -148,7 +148,6 @@ golang.org/x/exp/typeparams v0.0.0-20241009180824-f66d83c29e7c/go.mod h1:AbB0pIl golang.org/x/lint v0.0.0-20181026193005-c67002cb31c3/go.mod h1:UVdnD1Gm6xHRNCYTkRU2/jEulfH38KcIWyp/GAMgvoE= golang.org/x/lint v0.0.0-20190227174305-5b3e6a55c961/go.mod h1:wehouNa3lNwaWXcvxsM5YxQ5yQlVC4a0KAMCusXpPoU= golang.org/x/lint v0.0.0-20190313153728-d0100b6bd8b3/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc= -golang.org/x/mod v0.3.0/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA= golang.org/x/mod v0.4.2/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA= golang.org/x/mod v0.21.0 h1:vvrHzRwRfVKSiLrG+d4FMl/Qi4ukBCE6kZlTUkDYRT0= golang.org/x/mod v0.21.0/go.mod h1:6SkKJ3Xj0I0BrPOZoBy3bdMptDDU9oJrpohJ3eWZ1fY= @@ -158,7 +157,6 @@ golang.org/x/net v0.0.0-20190213061140-3a22650c66bd/go.mod h1:mL1N/T3taQHkDXs73r golang.org/x/net v0.0.0-20190311183353-d8887717615a/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg= golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg= golang.org/x/net v0.0.0-20190620200207-3b0461eec859/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s= -golang.org/x/net v0.0.0-20200822124328-c89045814202/go.mod h1:/O7V0waA8r7cgGh81Ro3o1hOxt32SMVPicZroKQ2sZA= golang.org/x/net v0.0.0-20201110031124-69a78807bb2b/go.mod h1:sp8m0HH+o8qH0wwXwYZr8TS3Oi6o0r6Gce1SSxlDquU= golang.org/x/net v0.0.0-20210405180319-a5a99cb37ef4/go.mod h1:p54w0d4576C0XHj96bSt6lcn1PtDYWL6XObtHCRCNQM= golang.org/x/net v0.30.0 h1:AcW1SDZMkb8IpzCdQUaIq2sP4sZ4zw+55h6ynffypl4= @@ -169,23 +167,23 @@ golang.org/x/oauth2 v0.23.0/go.mod h1:XYTD2NtWslqkgxebSiOHnXEap4TF09sJSc7H1sXbht golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= golang.org/x/sync v0.0.0-20181108010431-42b317875d0f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= -golang.org/x/sync v0.0.0-20200625203802-6e8e738ad208/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= golang.org/x/sync v0.0.0-20210220032951-036812b2e83c/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= golang.org/x/sync v0.8.0 h1:3NFvSEYkUoMifnESzZl15y791HH1qU2xm6eCJU5ZPXQ= golang.org/x/sync v0.8.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk= golang.org/x/sys v0.0.0-20180830151530-49385e6e1522/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY= golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY= golang.org/x/sys v0.0.0-20190412213103-97732733099d/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= -golang.org/x/sys v0.0.0-20200323222414-85ca7c5b95cd/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= golang.org/x/sys v0.0.0-20200930185726-fdedc70b468f/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= golang.org/x/sys v0.0.0-20210330210617-4fbd30eecc44/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= golang.org/x/sys v0.0.0-20210510120138-977fb7262007/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg= golang.org/x/sys v0.26.0 h1:KHjCJyddX0LoSTb3J+vWpupP9p0oznkqVk/IfjymZbo= golang.org/x/sys v0.26.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA= -golang.org/x/telemetry v0.0.0-20241028140143-9c0d19e65ba0 h1:od0RE4kmouF+x/o4zkTTSvBnGPZ2azGgCUmZdrbnEXM= -golang.org/x/telemetry v0.0.0-20241028140143-9c0d19e65ba0/go.mod h1:8nZWdGp9pq73ZI//QJyckMQab3yq7hoWi7SI0UIusVI= +golang.org/x/telemetry v0.0.0-20241106142447-58a1122356f5 h1:TCDqnvbBsFapViksHcHySl/sW4+rTGNIAoJJesHRuMM= +golang.org/x/telemetry v0.0.0-20241106142447-58a1122356f5/go.mod h1:8nZWdGp9pq73ZI//QJyckMQab3yq7hoWi7SI0UIusVI= golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo= +golang.org/x/term v0.25.0 h1:WtHI/ltw4NvSUig5KARz9h521QvRC8RmF/cuYqifU24= +golang.org/x/term v0.25.0/go.mod h1:RPyXicDX+6vLxogjjRxjgD2TKtmAO6NZBsBRfrOLu7M= golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ= golang.org/x/text v0.3.3/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ= golang.org/x/text v0.19.0 h1:kTxAhCbGbxhK0IwgSKiMO5awPoDQ0RpfiVYBfK860YM= @@ -198,7 +196,6 @@ golang.org/x/tools v0.0.0-20190226205152-f727befe758c/go.mod h1:9Yl7xja0Znq3iFh3 golang.org/x/tools v0.0.0-20190311212946-11955173bddd/go.mod h1:LCzVGOaR6xXOjkQ3onu1FJEFr0SW1gC7cKk1uF8kGRs= golang.org/x/tools v0.0.0-20190524140312-2c0ae7006135/go.mod h1:RgjU9mgBXZiqYHBnxXauZ1Gv1EHHAz9KjViQ78xBX0Q= golang.org/x/tools v0.0.0-20191119224855-298f0cb1881e/go.mod h1:b+2E5dAYhXwXZwtnZ6UAqBI28+e2cm9otk0dWdXHAEo= -golang.org/x/tools v0.0.0-20201007032633-0806396f153e/go.mod h1:z6u4i615ZeAfBE4XtMziQW1fSVJXACjjbWkB/mvPzlU= golang.org/x/tools v0.1.5/go.mod h1:o0xws9oXOQQZyjljx8fwUC0k7L1pTE6eaCbjGeHmOkk= golang.org/x/tools v0.26.0 h1:v/60pFQmzmT9ExmjDv2gGIfi3OqfKoEP6I5+umXlbnQ= golang.org/x/tools v0.26.0/go.mod h1:TPVVj70c7JJ3WCazhD8OdXcZg/og+b9+tH/KxylGwH0= @@ -208,17 +205,17 @@ golang.org/x/xerrors v0.0.0-20190717185122-a985d3407aa7/go.mod h1:I/5z698sn9Ka8T golang.org/x/xerrors v0.0.0-20191011141410-1b5146add898/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0= golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0= golang.org/x/xerrors v0.0.0-20200804184101-5ec99f83aff1/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0= -google.golang.org/api v0.203.0 h1:SrEeuwU3S11Wlscsn+LA1kb/Y5xT8uggJSkIhD08NAU= -google.golang.org/api v0.203.0/go.mod h1:BuOVyCSYEPwJb3npWvDnNmFI92f3GeRnHNkETneT3SI= +google.golang.org/api v0.205.0 h1:LFaxkAIpDb/GsrWV20dMMo5MR0h8UARTbn24LmD+0Pg= +google.golang.org/api v0.205.0/go.mod h1:NrK1EMqO8Xk6l6QwRAmrXXg2v6dzukhlOyvkYtnvUuc= google.golang.org/appengine v1.1.0/go.mod h1:EbEs0AVv82hx2wNQdGPgUI5lhzA/G0D9YwlJXL52JkM= google.golang.org/appengine v1.4.0/go.mod h1:xpcJRLb0r/rnEns0DIKYYv+WjYCduHsrkT7/EB5XEv4= google.golang.org/genproto v0.0.0-20180817151627-c66870c02cf8/go.mod h1:JiN7NxoALGmiZfu7CAH4rXhgtRTLTxftemlI0sWmxmc= google.golang.org/genproto v0.0.0-20190819201941-24fa4b261c55/go.mod h1:DMBHOl98Agz4BDEuKkezgsaosCRResVns1a3J2ZsMNc= google.golang.org/genproto v0.0.0-20200526211855-cb27e3aa2013/go.mod h1:NbSheEEYHJ7i3ixzK3sjbqSGDJWnxyFXZblF3eUsNvo= -google.golang.org/genproto/googleapis/api v0.0.0-20241021214115-324edc3d5d38 h1:2oV8dfuIkM1Ti7DwXc0BJfnwr9csz4TDXI9EmiI+Rbw= -google.golang.org/genproto/googleapis/api v0.0.0-20241021214115-324edc3d5d38/go.mod h1:vuAjtvlwkDKF6L1GQ0SokiRLCGFfeBUXWr/aFFkHACc= -google.golang.org/genproto/googleapis/rpc v0.0.0-20241021214115-324edc3d5d38 h1:zciRKQ4kBpFgpfC5QQCVtnnNAcLIqweL7plyZRQHVpI= -google.golang.org/genproto/googleapis/rpc v0.0.0-20241021214115-324edc3d5d38/go.mod h1:GX3210XPVPUjJbTUbvwI8f2IpZDMZuPJWDzDuebbviI= +google.golang.org/genproto/googleapis/api v0.0.0-20241104194629-dd2ea8efbc28 h1:M0KvPgPmDZHPlbRbaNU1APr28TvwvvdUPlSv7PUvy8g= +google.golang.org/genproto/googleapis/api v0.0.0-20241104194629-dd2ea8efbc28/go.mod h1:dguCy7UOdZhTvLzDyt15+rOrawrpM4q7DD9dQ1P11P4= +google.golang.org/genproto/googleapis/rpc v0.0.0-20241104194629-dd2ea8efbc28 h1:XVhgTWWV3kGQlwJHR3upFWZeTsei6Oks1apkZSeonIE= +google.golang.org/genproto/googleapis/rpc v0.0.0-20241104194629-dd2ea8efbc28/go.mod h1:GX3210XPVPUjJbTUbvwI8f2IpZDMZuPJWDzDuebbviI= google.golang.org/grpc v1.19.0/go.mod h1:mqu4LbDTu4XGKhr4mRzUsmM4RtVoemTSY81AxZiDr8c= google.golang.org/grpc v1.23.0/go.mod h1:Y5yQAOtifL1yxbo5wqy6BxZv8vAUGQwXBOALyacEbxg= google.golang.org/grpc v1.25.1/go.mod h1:c3i+UQWmh7LiEpx4sFZnkU36qjEYZ0imhYfXVyQciAY= @@ -246,7 +243,11 @@ honnef.co/go/tools v0.0.0-20190102054323-c2f93a96b099/go.mod h1:rf3lG4BRIbNafJWh honnef.co/go/tools v0.0.0-20190523083050-ea95bdfd59fc/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4= honnef.co/go/tools v0.5.1 h1:4bH5o3b5ZULQ4UrBmP+63W9r7qIkqJClEA9ko5YKx+I= honnef.co/go/tools v0.5.1/go.mod h1:e9irvo83WDG9/irijV44wr3tbhcFeRnfpVlRqVwpzMs= +mvdan.cc/editorconfig v0.3.0 h1:D1D2wLYEYGpawWT5SpM5pRivgEgXjtEXwC9MWhEY0gQ= +mvdan.cc/editorconfig v0.3.0/go.mod h1:NcJHuDtNOTEJ6251indKiWuzK6+VcrMuLzGMLKBFupQ= mvdan.cc/gofumpt v0.7.0 h1:bg91ttqXmi9y2xawvkuMXyvAA/1ZGJqYAEGjXuP0JXU= mvdan.cc/gofumpt v0.7.0/go.mod h1:txVFJy/Sc/mvaycET54pV8SW8gWxTlUuGHVEcncmNUo= +mvdan.cc/sh/v3 v3.10.0 h1:v9z7N1DLZ7owyLM/SXZQkBSXcwr2IGMm2LY2pmhVXj4= +mvdan.cc/sh/v3 v3.10.0/go.mod h1:z/mSSVyLFGZzqb3ZIKojjyqIx/xbmz/UHdCSv9HmqXY= mvdan.cc/unparam v0.0.0-20240917084806-57a3b4290ba3 h1:YkmTN1n5U60NM02j7TCSWRlW3fqNiuXe/eVXf0dLFN8= mvdan.cc/unparam v0.0.0-20240917084806-57a3b4290ba3/go.mod h1:z5yboO1sP1Q9pcfvS597TpfbNXQjphDlkCJHzt13ybc= diff --git a/internal/tools/tools.go b/internal/tools/tools.go index ff1c2b13..86ff7411 100644 --- a/internal/tools/tools.go +++ b/internal/tools/tools.go @@ -10,8 +10,8 @@ import ( _ "github.com/fzipp/gocyclo/cmd/gocyclo" _ "github.com/golangci/misspell/cmd/misspell" _ "github.com/gordonklaus/ineffassign" + _ "github.com/jstemmer/go-junit-report/v2" _ "github.com/kisielk/errcheck" - _ "github.com/kyoh86/looppointer" _ "github.com/securego/gosec/v2/cmd/gosec" _ "github.com/uudashr/gocognit/cmd/gocognit" _ "golang.org/x/tools/go/analysis/passes/nilness/cmd/nilness" @@ -19,5 +19,6 @@ import ( _ "golang.org/x/vuln/cmd/govulncheck" _ "honnef.co/go/tools/cmd/staticcheck" _ "mvdan.cc/gofumpt" + _ "mvdan.cc/sh/v3/cmd/shfmt" _ "mvdan.cc/unparam" ) diff --git a/internal/updater/check.go b/internal/updater/check.go index 12f4bece..84da6281 100644 --- a/internal/updater/check.go +++ b/internal/updater/check.go @@ -4,6 +4,7 @@ import ( "encoding/json" "fmt" "io" + "maps" "net/http" "slices" "time" @@ -12,7 +13,6 @@ import ( "github.com/AdguardTeam/golibs/errors" "github.com/AdguardTeam/golibs/ioutil" "github.com/AdguardTeam/golibs/log" - "golang.org/x/exp/maps" ) // TODO(a.garipov): Make configurable. @@ -120,8 +120,8 @@ func (u *Updater) downloadURL(versionObj map[string]string) (dlURL, key string, return dlURL, key, true } - keys := maps.Keys(versionObj) - slices.Sort(keys) + keys := slices.Sorted(maps.Keys(versionObj)) + log.Error("updater: key %q not found; got keys %q", key, keys) return "", key, false diff --git a/scripts/blocked-services/main.go b/scripts/blocked-services/main.go index 43ac4fae..bdf7c17a 100644 --- a/scripts/blocked-services/main.go +++ b/scripts/blocked-services/main.go @@ -16,6 +16,7 @@ import ( "text/template" "time" + "github.com/AdguardTeam/golibs/errors" "github.com/AdguardTeam/golibs/logutil/slogutil" ) @@ -30,14 +31,13 @@ func main() { // Validate the URL. _, err := url.Parse(urlStr) - check(err) + errors.Check(err) c := &http.Client{ Timeout: 10 * time.Second, } - resp, err := c.Get(urlStr) - check(err) + resp := errors.Must(c.Get(urlStr)) defer slogutil.CloseAndLog(ctx, l, resp.Body, slog.LevelError) if resp.StatusCode != http.StatusOK { @@ -46,7 +46,7 @@ func main() { hlSvcs := &hlServices{} err = json.NewDecoder(resp.Body).Decode(hlSvcs) - check(err) + errors.Check(err) // Sort all services and rules to make the output more predictable. slices.SortStableFunc(hlSvcs.BlockedServices, func(a, b *hlServicesService) (res int) { @@ -59,20 +59,21 @@ func main() { // Use another set of delimiters to prevent them interfering with the Go // code. tmpl, err := template.New("main").Delims("<%", "%>").Funcs(template.FuncMap{ - "isnotlast": func(idx, sliceLen int) (ok bool) { return idx != sliceLen-1 }, + "isnotlast": func(idx, sliceLen int) (ok bool) { + return idx != sliceLen-1 + }, }).Parse(tmplStr) - check(err) + errors.Check(err) f, err := os.OpenFile( "./internal/filtering/servicelist.go", os.O_CREATE|os.O_TRUNC|os.O_WRONLY, 0o644, ) - check(err) + errors.Check(err) defer slogutil.CloseAndLog(ctx, l, f, slog.LevelError) - err = tmpl.Execute(f, hlSvcs) - check(err) + errors.Check(tmpl.Execute(f, hlSvcs)) } // tmplStr is the template for the Go source file with the services. @@ -100,13 +101,6 @@ var blockedServices = []blockedService{<% $l := len .BlockedServices %> }<% if isnotlast $i $l %>, <% end %><% end %>} ` -// check is a simple error-checking helper for scripts. -func check(err error) { - if err != nil { - panic(err) - } -} - // hlServices is the JSON structure for the Hostlists Registry blocked service // index. type hlServices struct { diff --git a/scripts/hooks/pre-commit b/scripts/hooks/pre-commit index 307f3ec6..c3fe750a 100755 --- a/scripts/hooks/pre-commit +++ b/scripts/hooks/pre-commit @@ -2,81 +2,78 @@ set -e -f -u -# This comment is used to simplify checking local copies of the script. -# Bump this number every time a significant change is made to this -# script. +# This comment is used to simplify checking local copies of the script. Bump +# this number every time a significant change is made to this script. # -# AdGuard-Project-Version: 3 +# AdGuard-Project-Version: 5 # TODO(a.garipov): Add pre-merge-commit. -# Only show interactive prompts if there a terminal is attached to -# stdout. While this technically doesn't guarantee that reading from -# /dev/tty works, this should work reasonably well on all of our -# supported development systems and in most terminal emulators. +# Only show interactive prompts if there a terminal is attached to stdout. +# While this technically doesn't guarantee that reading from /dev/tty works, +# this should work reasonably well on all of our supported development systems +# and in most terminal emulators. is_tty='0' -if [ -t '1' ] -then +if [ -t '1' ]; then is_tty='1' fi readonly is_tty -# prompt is a helper that prompts the user for interactive input if that -# can be done. If there is no terminal attached, it sleeps for two -# seconds, giving the programmer some time to react, and returns with -# a zero exit code. +# prompt is a helper that prompts the user for interactive input if that can be +# done. If there is no terminal attached, it sleeps for two seconds, giving the +# programmer some time to react, and returns with a zero exit code. prompt() { - if [ "$is_tty" -eq '0' ] - then + if [ "$is_tty" -eq '0' ]; then sleep 2 return 0 fi - while true - do + while true; do printf 'commit anyway? y/[n]: ' - read -r ans < /dev/tty + read -r ans &2 exit 1 ;; @@ -64,14 +61,13 @@ esac # Check VERSION against the default value from the Makefile. If it is that, use # the version calculation script. version="${VERSION:-}" -if [ "$version" = 'v0.0.0' ] || [ "$version" = '' ] -then - version="$( sh ./scripts/make/version.sh )" +if [ "$version" = 'v0.0.0' ] || [ "$version" = '' ]; then + version="$(sh ./scripts/make/version.sh)" fi readonly version # Set date and time of the latest commit unless already set. -committime="${SOURCE_DATE_EPOCH:-$( git log -1 --pretty=%ct )}" +committime="${SOURCE_DATE_EPOCH:-$(git log -1 --pretty=%ct)}" readonly committime # Set the linker flags accordingly: set the release channel and the current @@ -84,11 +80,9 @@ ldflags="-s -w" ldflags="${ldflags} -X ${version_pkg}.version=${version}" ldflags="${ldflags} -X ${version_pkg}.channel=${channel}" ldflags="${ldflags} -X ${version_pkg}.committime=${committime}" -if [ "${GOARM:-}" != '' ] -then +if [ "${GOARM:-}" != '' ]; then ldflags="${ldflags} -X ${version_pkg}.goarm=${GOARM}" -elif [ "${GOMIPS:-}" != '' ] -then +elif [ "${GOMIPS:-}" != '' ]; then ldflags="${ldflags} -X ${version_pkg}.gomips=${GOMIPS}" fi readonly ldflags @@ -99,9 +93,8 @@ readonly parallelism # Use GOFLAGS for -p, because -p=0 simply disables the build instead of leaving # the default value. -if [ "${parallelism}" != '' ] -then - GOFLAGS="${GOFLAGS:-} -p=${parallelism}" +if [ "${parallelism}" != '' ]; then + GOFLAGS="${GOFLAGS:-} -p=${parallelism}" fi readonly GOFLAGS export GOFLAGS @@ -115,8 +108,7 @@ readonly o_flags # Allow users to enable the race detector. Unfortunately, that means that cgo # must be enabled. -if [ "${RACE:-0}" -eq '0' ] -then +if [ "${RACE:-0}" -eq '0' ]; then CGO_ENABLED='0' race_flags='--race=0' else @@ -130,24 +122,23 @@ GO111MODULE='on' export GO111MODULE # Build the new binary if requested. -if [ "${NEXTAPI:-0}" -eq '0' ] -then +if [ "${NEXTAPI:-0}" -eq '0' ]; then tags_flags='--tags=' else tags_flags='--tags=next' fi readonly tags_flags -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then "$go" env fi -"$go" build\ - --ldflags="$ldflags"\ - "$race_flags"\ - "$tags_flags"\ - --trimpath\ - "$o_flags"\ - "$v_flags"\ - "$x_flags" +"$go" build \ + --ldflags="$ldflags" \ + "$race_flags" \ + "$tags_flags" \ + --trimpath \ + "$o_flags" \ + "$v_flags" \ + "$x_flags" \ + ; diff --git a/scripts/make/go-deps.sh b/scripts/make/go-deps.sh index f12d2df7..df9a7df0 100644 --- a/scripts/make/go-deps.sh +++ b/scripts/make/go-deps.sh @@ -3,18 +3,16 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a significant change is made to this script. # -# AdGuard-Project-Version: 1 +# AdGuard-Project-Version: 2 verbose="${VERBOSE:-0}" readonly verbose -if [ "$verbose" -gt '1' ] -then +if [ "$verbose" -gt '1' ]; then env set -x x_flags='-x=1' -elif [ "$verbose" -gt '0' ] -then +elif [ "$verbose" -gt '0' ]; then set -x x_flags='-x=0' else diff --git a/scripts/make/go-fuzz.sh b/scripts/make/go-fuzz.sh index 8f10b4f1..41561de8 100644 --- a/scripts/make/go-fuzz.sh +++ b/scripts/make/go-fuzz.sh @@ -7,13 +7,11 @@ readonly verbose # 0 = Don't print anything except for errors. # 1 = Print commands, but not nested commands. # 2 = Print everything. -if [ "$verbose" -gt '1' ] -then +if [ "$verbose" -gt '1' ]; then set -x v_flags='-v=1' x_flags='-x=1' -elif [ "$verbose" -gt '0' ] -then +elif [ "$verbose" -gt '0' ]; then set -x v_flags='-v=1' x_flags='-x=0' @@ -26,8 +24,7 @@ readonly v_flags x_flags set -e -f -u -if [ "${RACE:-1}" -eq '0' ] -then +if [ "${RACE:-1}" -eq '0' ]; then race_flags='--race=0' else race_flags='--race=1' @@ -44,15 +41,15 @@ fuzztime_flags="${FUZZTIME_FLAGS:---fuzztime=20s}" readonly go count_flags shuffle_flags timeout_flags fuzztime_flags # TODO(a.garipov): File an issue about using --fuzz with multiple packages. -"$go" test\ - "$count_flags"\ - "$shuffle_flags"\ - "$race_flags"\ - "$timeout_flags"\ - "$x_flags"\ - "$v_flags"\ - "$fuzztime_flags"\ - --fuzz='.'\ - --run='^$'\ - ./internal/filtering/rulelist/\ +"$go" test \ + "$count_flags" \ + "$shuffle_flags" \ + "$race_flags" \ + "$timeout_flags" \ + "$x_flags" \ + "$v_flags" \ + "$fuzztime_flags" \ + --fuzz='.' \ + --run='^$' \ + ./internal/filtering/rulelist/ \ ; diff --git a/scripts/make/go-lint.sh b/scripts/make/go-lint.sh index 41f70d76..3fbbc62a 100644 --- a/scripts/make/go-lint.sh +++ b/scripts/make/go-lint.sh @@ -3,19 +3,17 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a significant change is made to this script. # -# AdGuard-Project-Version: 8 +# AdGuard-Project-Version: 13 verbose="${VERBOSE:-0}" readonly verbose -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi # Set $EXIT_ON_ERROR to zero to see all errors. -if [ "${EXIT_ON_ERROR:-1}" -eq '0' ] -then +if [ "${EXIT_ON_ERROR:-1}" -eq '0' ]; then set +e else set -e @@ -23,13 +21,9 @@ fi set -f -u - - # Source the common helpers, including not_found and run_linter. . ./scripts/make/helper.sh - - # Simple analyzers # blocklist_imports is a simple check against unwanted packages. The following @@ -38,8 +32,15 @@ set -f -u # * Package errors is replaced by our own package in the # github.com/AdguardTeam/golibs module. # -# * Packages golang.org/x/exp/slices and golang.org/x/net/context have been -# moved into stdlib. +# * Packages log and github.com/AdguardTeam/golibs/log are replaced by +# stdlib's new package log/slog and AdGuard's new utilities package +# github.com/AdguardTeam/golibs/logutil/slogutil. +# +# * Package github.com/prometheus/client_golang/prometheus/promauto is not +# recommended, as it encourages reliance on global state. +# +# * Packages golang.org/x/exp/maps, golang.org/x/exp/slices, and +# golang.org/x/net/context have been moved into stdlib. # # * Package io/ioutil is soft-deprecated. # @@ -54,47 +55,55 @@ set -f -u # # * Package unsafe is… unsafe. # -# If your project needs more exceptions, add and document them. Currently, -# there are only two standard exceptions: +# Currently, the only standard exception are files generated from protobuf +# schemas, which use package reflect. If your project needs more exceptions, +# add and document them. # -# * Files generated from protobuf schemas, which use package reflect. +# NOTE: Flag -H for grep is non-POSIX but all of Busybox, GNU, macOS, and +# OpenBSD support it. # -# * Windows-specific code caused by golang.org/x/sys/windows API design. +# NOTE: Exclude the permission_windows.go, because it requires unsafe for the +# OS APIs. # # TODO(a.garipov): Add golibs/log. -# -# TODO(a.garipov): Add deprecated package golang.org/x/exp/maps once all -# projects switch to Go 1.23. blocklist_imports() { - git grep\ - -e '[[:space:]]"errors"$'\ - -e '[[:space:]]"golang.org/x/exp/slices"$'\ - -e '[[:space:]]"golang.org/x/net/context"$'\ - -e '[[:space:]]"io/ioutil"$'\ - -e '[[:space:]]"log"$'\ - -e '[[:space:]]"reflect"$'\ - -e '[[:space:]]"sort"$'\ - -e '[[:space:]]"unsafe"$'\ - -n\ - -- '*.go'\ - ':!*.pb.go'\ - ':!./internal/aghos/permission_windows.go'\ - | sed -e 's/^\([^[:space:]]\+\)\(.*\)$/\1 blocked import:\2/'\ - || exit 0 + find . \ + -type 'f' \ + -name '*.go' \ + '!' '(' \ + -name '*.pb.go' \ + -o -path './internal/aghos/permission_windows.go' \ + ')' \ + -exec \ + 'grep' \ + '-H' \ + '-e' '[[:space:]]"errors"$' \ + '-e' '[[:space:]]"github.com/prometheus/client_golang/prometheus/promauto"$' \ + '-e' '[[:space:]]"golang.org/x/exp/maps"$' \ + '-e' '[[:space:]]"golang.org/x/exp/slices"$' \ + '-e' '[[:space:]]"golang.org/x/net/context"$' \ + '-e' '[[:space:]]"io/ioutil"$' \ + '-e' '[[:space:]]"log"$' \ + '-e' '[[:space:]]"reflect"$' \ + '-e' '[[:space:]]"sort"$' \ + '-e' '[[:space:]]"unsafe"$' \ + '-n' \ + '{}' \ + ';' } # method_const is a simple check against the usage of some raw strings and # numbers where one should use named constants. method_const() { - git grep -F\ - -e '"DELETE"'\ - -e '"GET"'\ - -e '"PATCH"'\ - -e '"POST"'\ - -e '"PUT"'\ - -n\ - -- '*.go'\ - | sed -e 's/^\([^[:space:]]\+\)\(.*\)$/\1 http method literal:\2/'\ + git grep -F \ + -e '"DELETE"' \ + -e '"GET"' \ + -e '"PATCH"' \ + -e '"POST"' \ + -e '"PUT"' \ + -n \ + -- '*.go' \ + | sed -e 's/^\([^[:space:]]\+\)\(.*\)$/\1 http method literal:\2/' \ || exit 0 } @@ -103,26 +112,25 @@ method_const() { # use of filenames like client_manager.go. underscores() { underscore_files="$( - git ls-files '*_*.go'\ - | grep -F\ - -e '_bsd.go'\ - -e '_darwin.go'\ - -e '_freebsd.go'\ - -e '_generate.go'\ - -e '_linux.go'\ - -e '_next.go'\ - -e '_openbsd.go'\ - -e '_others.go'\ - -e '_test.go'\ - -e '_unix.go'\ - -e '_windows.go'\ - -v\ + git ls-files '*_*.go' \ + | grep -F \ + -e '_bsd.go' \ + -e '_darwin.go' \ + -e '_freebsd.go' \ + -e '_generate.go' \ + -e '_linux.go' \ + -e '_next.go' \ + -e '_openbsd.go' \ + -e '_others.go' \ + -e '_test.go' \ + -e '_unix.go' \ + -e '_windows.go' \ + -v \ | sed -e 's/./\t\0/' )" readonly underscore_files - if [ "$underscore_files" != '' ] - then + if [ "$underscore_files" != '' ]; then echo 'found file names with underscores:' echo "$underscore_files" fi @@ -130,10 +138,6 @@ underscores() { # TODO(a.garipov): Add an analyzer to look for `fallthrough`, `goto`, and `new`? - - -# Checks - run_linter -e blocklist_imports run_linter -e method_const @@ -151,129 +155,129 @@ run_linter govulncheck ./... run_linter gocyclo --over 10 . # TODO(a.garipov): Enable 10 for all. -run_linter gocognit --over='20'\ - ./internal/querylog/\ +run_linter gocognit --over='20' \ + ./internal/querylog/ \ ; -run_linter gocognit --over='19'\ - ./internal/home/\ +run_linter gocognit --over='19' \ + ./internal/home/ \ ; -run_linter gocognit --over='18'\ - ./internal/aghtls/\ +run_linter gocognit --over='18' \ + ./internal/aghtls/ \ ; -run_linter gocognit --over='15'\ - ./internal/aghos/\ - ./internal/filtering/\ +run_linter gocognit --over='15' \ + ./internal/aghos/ \ + ./internal/filtering/ \ ; -run_linter gocognit --over='14'\ - ./internal/dhcpd\ +run_linter gocognit --over='14' \ + ./internal/dhcpd \ ; -run_linter gocognit --over='13'\ - ./internal/aghnet/\ +run_linter gocognit --over='13' \ + ./internal/aghnet/ \ ; -run_linter gocognit --over='12'\ - ./internal/filtering/rewrite/\ +run_linter gocognit --over='12' \ + ./internal/filtering/rewrite/ \ ; -run_linter gocognit --over='11'\ - ./internal/updater/\ +run_linter gocognit --over='11' \ + ./internal/updater/ \ ; -run_linter gocognit --over='10'\ - ./internal/aghalg/\ - ./internal/aghhttp/\ - ./internal/aghrenameio/\ - ./internal/aghtest/\ - ./internal/arpdb/\ - ./internal/client/\ - ./internal/configmigrate/\ - ./internal/dhcpsvc\ - ./internal/dnsforward/\ - ./internal/filtering/hashprefix/\ - ./internal/filtering/rulelist/\ - ./internal/filtering/safesearch/\ - ./internal/ipset\ - ./internal/next/\ - ./internal/rdns/\ - ./internal/schedule/\ - ./internal/stats/\ - ./internal/tools/\ - ./internal/version/\ - ./internal/whois/\ - ./scripts/\ +run_linter gocognit --over='10' \ + ./internal/aghalg/ \ + ./internal/aghhttp/ \ + ./internal/aghrenameio/ \ + ./internal/aghtest/ \ + ./internal/arpdb/ \ + ./internal/client/ \ + ./internal/configmigrate/ \ + ./internal/dhcpsvc \ + ./internal/dnsforward/ \ + ./internal/filtering/hashprefix/ \ + ./internal/filtering/rulelist/ \ + ./internal/filtering/safesearch/ \ + ./internal/ipset \ + ./internal/next/ \ + ./internal/rdns/ \ + ./internal/schedule/ \ + ./internal/stats/ \ + ./internal/tools/ \ + ./internal/version/ \ + ./internal/whois/ \ + ./scripts/ \ ; run_linter ineffassign ./... run_linter unparam ./... -git ls-files -- 'Makefile' '*.conf' '*.go' '*.mod' '*.sh' '*.yaml' '*.yml'\ - | xargs misspell --error\ +git ls-files -- 'Makefile' '*.conf' '*.go' '*.mod' '*.sh' '*.yaml' '*.yml' \ + | xargs misspell --error \ | sed -e 's/^/misspell: /' run_linter nilness ./... # TODO(a.garipov): Enable for all. run_linter fieldalignment \ - ./internal/aghalg/\ - ./internal/aghhttp/\ - ./internal/aghos/\ - ./internal/aghrenameio/\ - ./internal/aghtest/\ - ./internal/aghtls/\ - ./internal/arpdb/\ - ./internal/client/\ - ./internal/configmigrate/\ - ./internal/dhcpsvc/\ - ./internal/filtering/hashprefix/\ - ./internal/filtering/rewrite/\ - ./internal/filtering/rulelist/\ - ./internal/filtering/safesearch/\ - ./internal/ipset/\ - ./internal/next/...\ - ./internal/querylog/\ - ./internal/rdns/\ - ./internal/schedule/\ - ./internal/stats/\ - ./internal/updater/\ - ./internal/version/\ - ./internal/whois/\ + ./internal/aghalg/ \ + ./internal/aghhttp/ \ + ./internal/aghos/ \ + ./internal/aghrenameio/ \ + ./internal/aghtest/ \ + ./internal/aghtls/ \ + ./internal/arpdb/ \ + ./internal/client/ \ + ./internal/configmigrate/ \ + ./internal/dhcpsvc/ \ + ./internal/filtering/hashprefix/ \ + ./internal/filtering/rewrite/ \ + ./internal/filtering/rulelist/ \ + ./internal/filtering/safesearch/ \ + ./internal/ipset/ \ + ./internal/next/... \ + ./internal/querylog/ \ + ./internal/rdns/ \ + ./internal/schedule/ \ + ./internal/stats/ \ + ./internal/updater/ \ + ./internal/version/ \ + ./internal/whois/ \ ; run_linter -e shadow --strict ./... # TODO(a.garipov): Enable for all. # TODO(e.burkov): Re-enable G115. -run_linter gosec --exclude G115 --quiet\ - ./internal/aghalg/\ - ./internal/aghhttp/\ - ./internal/aghnet/\ - ./internal/aghos/\ - ./internal/aghrenameio/\ - ./internal/aghtest/\ - ./internal/arpdb/\ - ./internal/client/\ - ./internal/configmigrate/\ - ./internal/dhcpd/\ - ./internal/dhcpsvc/\ - ./internal/dnsforward/\ - ./internal/filtering/hashprefix/\ - ./internal/filtering/rewrite/\ - ./internal/filtering/rulelist/\ - ./internal/filtering/safesearch/\ - ./internal/ipset/\ - ./internal/next/\ - ./internal/rdns/\ - ./internal/schedule/\ - ./internal/stats/\ - ./internal/tools/\ - ./internal/version/\ - ./internal/whois/\ +run_linter gosec --exclude G115 --quiet \ + ./internal/aghalg/ \ + ./internal/aghhttp/ \ + ./internal/aghnet/ \ + ./internal/aghos/ \ + ./internal/aghrenameio/ \ + ./internal/aghtest/ \ + ./internal/arpdb/ \ + ./internal/client/ \ + ./internal/configmigrate/ \ + ./internal/dhcpd/ \ + ./internal/dhcpsvc/ \ + ./internal/dnsforward/ \ + ./internal/filtering/hashprefix/ \ + ./internal/filtering/rewrite/ \ + ./internal/filtering/rulelist/ \ + ./internal/filtering/safesearch/ \ + ./internal/ipset/ \ + ./internal/next/ \ + ./internal/rdns/ \ + ./internal/schedule/ \ + ./internal/stats/ \ + ./internal/tools/ \ + ./internal/version/ \ + ./internal/whois/ \ ; run_linter errcheck ./... diff --git a/scripts/make/go-test.sh b/scripts/make/go-test.sh index 3727587e..901e69f4 100644 --- a/scripts/make/go-test.sh +++ b/scripts/make/go-test.sh @@ -3,7 +3,7 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a significant change is made to this script. # -# AdGuard-Project-Version: 1 +# AdGuard-Project-Version: 6 verbose="${VERBOSE:-0}" readonly verbose @@ -12,13 +12,11 @@ readonly verbose # 0 = Don't print anything except for errors. # 1 = Print commands, but not nested commands. # 2 = Print everything. -if [ "$verbose" -gt '1' ] -then +if [ "$verbose" -gt '1' ]; then set -x v_flags='-v=1' x_flags='-x=1' -elif [ "$verbose" -gt '0' ] -then +elif [ "$verbose" -gt '0' ]; then set -x v_flags='-v=1' x_flags='-x=0' @@ -31,29 +29,54 @@ readonly v_flags x_flags set -e -f -u -if [ "${RACE:-1}" -eq '0' ] -then +if [ "${RACE:-1}" -eq '0' ]; then race_flags='--race=0' else race_flags='--race=1' fi readonly race_flags +count_flags='--count=2' +cover_flags='--coverprofile=./cover.out' go="${GO:-go}" -readonly go - -count_flags='--count=1' -cover_flags='--coverprofile=./coverage.txt' shuffle_flags='--shuffle=on' timeout_flags="${TIMEOUT_FLAGS:---timeout=90s}" -readonly count_flags cover_flags shuffle_flags timeout_flags +readonly count_flags cover_flags go shuffle_flags timeout_flags -"$go" test\ - "$count_flags"\ - "$cover_flags"\ - "$race_flags"\ - "$shuffle_flags"\ - "$timeout_flags"\ - "$v_flags"\ - "$x_flags"\ - ./... +go_test() { + "$go" test \ + "$count_flags" \ + "$cover_flags" \ + "$race_flags" \ + "$shuffle_flags" \ + "$timeout_flags" \ + "$v_flags" \ + "$x_flags" \ + ./... +} + +test_reports_dir="${TEST_REPORTS_DIR:-}" +readonly test_reports_dir + +if [ "$test_reports_dir" = '' ]; then + go_test + + exit "$?" +fi + +mkdir -p "$test_reports_dir" + +# NOTE: The pipe ignoring the exit code here is intentional, as go-junit-report +# will set the exit code to be saved. +go_test 2>&1 \ + | tee "${test_reports_dir}/test-output.txt" + +# Don't fail on errors in exporting, because TEST_REPORTS_DIR is generally only +# not empty in CI, and so the exit code must be preserved to exit with it later. +set +e +go-junit-report \ + --in "${test_reports_dir}/test-output.txt" \ + --set-exit-code \ + >"${test_reports_dir}/test-report.xml" +printf '%s\n' "$?" \ + >"${test_reports_dir}/test-exit-code.txt" diff --git a/scripts/make/go-tools.sh b/scripts/make/go-tools.sh index 295659c9..d4ed6586 100644 --- a/scripts/make/go-tools.sh +++ b/scripts/make/go-tools.sh @@ -3,18 +3,16 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a significant change is made to this script. # -# AdGuard-Project-Version: 4 +# AdGuard-Project-Version: 6 verbose="${VERBOSE:-0}" readonly verbose -if [ "$verbose" -gt '1' ] -then +if [ "$verbose" -gt '1' ]; then set -x v_flags='-v=1' x_flags='-x=1' -elif [ "$verbose" -gt '0' ] -then +elif [ "$verbose" -gt '0' ]; then set -x v_flags='-v=1' x_flags='-x=0' @@ -33,45 +31,49 @@ readonly go # Remove only the actual binaries in the bin/ directory, as developers may add # their own scripts there. Most commonly, a script named “go” for tools that # call the go binary and need a particular version. -rm -f\ - bin/errcheck\ - bin/fieldalignment\ - bin/gocognit\ - bin/gocyclo\ - bin/gofumpt\ - bin/gosec\ - bin/govulncheck\ - bin/ineffassign\ - bin/misspell\ - bin/nilness\ - bin/shadow\ - bin/staticcheck\ - bin/unparam\ +rm -f \ + bin/errcheck \ + bin/fieldalignment \ + bin/go-junit-report \ + bin/gocognit \ + bin/gocyclo \ + bin/gofumpt \ + bin/gosec \ + bin/govulncheck \ + bin/ineffassign \ + bin/misspell \ + bin/nilness \ + bin/shadow \ + bin/shfmt \ + bin/staticcheck \ + bin/unparam \ ; # Reset GOARCH and GOOS to make sure we install the tools for the native # architecture even when we're cross-compiling the main binary, and also to # prevent the "cannot install cross-compiled binaries when GOBIN is set" error. -env\ - GOARCH=""\ - GOBIN="${PWD}/bin"\ - GOOS=""\ - GOWORK='off'\ - "$go" install\ - --modfile=./internal/tools/go.mod\ - "$v_flags"\ - "$x_flags"\ - github.com/fzipp/gocyclo/cmd/gocyclo\ - github.com/golangci/misspell/cmd/misspell\ - github.com/gordonklaus/ineffassign\ - github.com/kisielk/errcheck\ - github.com/securego/gosec/v2/cmd/gosec\ - github.com/uudashr/gocognit/cmd/gocognit\ - golang.org/x/tools/go/analysis/passes/fieldalignment/cmd/fieldalignment\ - golang.org/x/tools/go/analysis/passes/nilness/cmd/nilness\ - golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow\ - golang.org/x/vuln/cmd/govulncheck\ - honnef.co/go/tools/cmd/staticcheck\ - mvdan.cc/gofumpt\ - mvdan.cc/unparam\ +env \ + GOARCH="" \ + GOBIN="${PWD}/bin" \ + GOOS="" \ + GOWORK='off' \ + "$go" install \ + --modfile=./internal/tools/go.mod \ + "$v_flags" \ + "$x_flags" \ + github.com/fzipp/gocyclo/cmd/gocyclo \ + github.com/golangci/misspell/cmd/misspell \ + github.com/gordonklaus/ineffassign \ + github.com/jstemmer/go-junit-report/v2 \ + github.com/kisielk/errcheck \ + github.com/securego/gosec/v2/cmd/gosec \ + github.com/uudashr/gocognit/cmd/gocognit \ + golang.org/x/tools/go/analysis/passes/fieldalignment/cmd/fieldalignment \ + golang.org/x/tools/go/analysis/passes/nilness/cmd/nilness \ + golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow \ + golang.org/x/vuln/cmd/govulncheck \ + honnef.co/go/tools/cmd/staticcheck \ + mvdan.cc/gofumpt \ + mvdan.cc/sh/v3/cmd/shfmt \ + mvdan.cc/unparam \ ; diff --git a/scripts/make/go-upd-tools.sh b/scripts/make/go-upd-tools.sh index 787475ac..de4b1307 100644 --- a/scripts/make/go-upd-tools.sh +++ b/scripts/make/go-upd-tools.sh @@ -3,18 +3,16 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a significant change is made to this script. # -# AdGuard-Project-Version: 2 +# AdGuard-Project-Version: 3 verbose="${VERBOSE:-0}" readonly verbose -if [ "$verbose" -gt '1' ] -then +if [ "$verbose" -gt '1' ]; then env set -x x_flags='-x=1' -elif [ "$verbose" -gt '0' ] -then +elif [ "$verbose" -gt '0' ]; then set -x x_flags='-x=0' else diff --git a/scripts/make/helper.sh b/scripts/make/helper.sh index 9c4ff88b..8caa0477 100644 --- a/scripts/make/helper.sh +++ b/scripts/make/helper.sh @@ -8,9 +8,7 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a remarkable change is made to this script. # -# AdGuard-Project-Version: 3 - - +# AdGuard-Project-Version: 4 # Deferred helpers @@ -23,8 +21,7 @@ make sure you have installed the linter binaries using: readonly not_found_msg not_found() { - if [ "$?" -eq '127' ] - then + if [ "$?" -eq '127' ]; then # Code 127 is the exit status a shell uses when a command or a file is # not found, according to the Bash Hackers wiki. # @@ -34,8 +31,6 @@ not_found() { } trap not_found EXIT - - # Helpers # run_linter runs the given linter with two additions: @@ -47,8 +42,7 @@ trap not_found EXIT run_linter() ( set +e - if [ "${VERBOSE:-0}" -lt '2' ] - then + if [ "${VERBOSE:-0}" -lt '2' ]; then set +x fi @@ -56,8 +50,7 @@ run_linter() ( shift exit_on_output='0' - if [ "$cmd" = '-e' ] - then + if [ "$cmd" = '-e' ]; then exit_on_output='1' cmd="${1:?run_linter: provide a command}" shift @@ -65,17 +58,15 @@ run_linter() ( readonly cmd - output="$( "$cmd" "$@" )" + output="$("$cmd" "$@")" exitcode="$?" readonly output - if [ "$output" != '' ] - then + if [ "$output" != '' ]; then echo "$output" | sed -e "s/^/${cmd}: /" - if [ "$exitcode" -eq '0' ] && [ "$exit_on_output" -eq '1' ] - then + if [ "$exitcode" -eq '0' ] && [ "$exit_on_output" -eq '1' ]; then exitcode='1' fi fi diff --git a/scripts/make/md-lint.sh b/scripts/make/md-lint.sh index 4e2ca062..e28623f0 100644 --- a/scripts/make/md-lint.sh +++ b/scripts/make/md-lint.sh @@ -3,21 +3,18 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a remarkable change is made to this script. # -# AdGuard-Project-Version: 2 +# AdGuard-Project-Version: 3 verbose="${VERBOSE:-0}" readonly verbose set -e -f -u -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi -# NOTE: Adjust for your project. -# markdownlint\ -# ./README.md\ -# ; - # TODO(e.burkov): Lint markdown documents within this project. +# markdownlint \ +# ./README.md \ +# ; diff --git a/scripts/make/sh-lint.sh b/scripts/make/sh-lint.sh index 83ed6f1d..ea3c52e6 100644 --- a/scripts/make/sh-lint.sh +++ b/scripts/make/sh-lint.sh @@ -3,7 +3,7 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a remarkable change is made to this script. # -# AdGuard-Project-Version: 2 +# AdGuard-Project-Version: 3 verbose="${VERBOSE:-0}" readonly verbose @@ -11,16 +11,23 @@ readonly verbose # Don't use -f, because we use globs in this script. set -e -u -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi -# NOTE: Adjust for your project. -# -# TODO(e.burkov): Add build-docker.sh, build-release.sh and install.sh. -shellcheck -e 'SC2250' -f 'gcc' -o 'all' -x --\ - ./scripts/hooks/*\ - ./scripts/snap/*\ - ./scripts/make/*\ +# Source the common helpers, including not_found and run_linter. +. ./scripts/make/helper.sh + +run_linter -e shfmt --binary-next-line -d -p -s \ + ./scripts/hooks/* \ + ./scripts/make/*.sh \ + ./scripts/snap/*.sh \ + ./snap/local/*.sh \ + ; + +shellcheck -e 'SC2250' -f 'gcc' -o 'all' -x -- \ + ./scripts/hooks/* \ + ./scripts/make/*.sh \ + ./scripts/snap/*.sh \ + ./snap/local/*.sh \ ; diff --git a/scripts/make/txt-lint.sh b/scripts/make/txt-lint.sh index 578297ec..3f980085 100644 --- a/scripts/make/txt-lint.sh +++ b/scripts/make/txt-lint.sh @@ -3,19 +3,17 @@ # This comment is used to simplify checking local copies of the script. Bump # this number every time a remarkable change is made to this script. # -# AdGuard-Project-Version: 5 +# AdGuard-Project-Version: 8 verbose="${VERBOSE:-0}" readonly verbose -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi # Set $EXIT_ON_ERROR to zero to see all errors. -if [ "${EXIT_ON_ERROR:-1}" -eq '0' ] -then +if [ "${EXIT_ON_ERROR:-1}" -eq '0' ]; then set +e else set -e @@ -32,19 +30,30 @@ set -f -u # trailing_newlines is a simple check that makes sure that all plain-text files # have a trailing newlines to make sure that all tools work correctly with them. trailing_newlines() ( - nl="$( printf "\n" )" + nl="$(printf '\n')" readonly nl - # NOTE: Adjust for your project. - git ls-files\ - ':!*.png'\ - ':!*.tar.gz'\ - ':!*.zip'\ - | while read -r f - do - final_byte="$( tail -c -1 "$f" )" - if [ "$final_byte" != "$nl" ] - then + find . \ + -type 'f' \ + '!' '(' \ + -name '*.db' \ + -o -name '*.exe' \ + -o -name '*.out' \ + -o -name '*.png' \ + -o -name '*.svg' \ + -o -name '*.tar.gz' \ + -o -name '*.test' \ + -o -name '*.zip' \ + -o -name 'AdGuardHome' \ + -o -name 'adguard-home' \ + -o -path '*/node_modules/*' \ + -o -path './.git/*' \ + -o -path './bin/*' \ + -o -path './build/*' \ + ')' \ + | while read -r f; do + final_byte="$(tail -c -1 "$f")" + if [ "$final_byte" != "$nl" ]; then printf '%s: must have a trailing newline\n' "$f" fi done @@ -53,19 +62,26 @@ trailing_newlines() ( # trailing_whitespace is a simple check that makes sure that there are no # trailing whitespace in plain-text files. trailing_whitespace() { - # NOTE: Adjust for your project. - git ls-files\ - ':!*.bmp'\ - ':!*.jpg'\ - ':!*.mmdb'\ - ':!*.png'\ - ':!*.svg'\ - ':!*.tar.gz'\ - ':!*.webp'\ - ':!*.zip'\ - | while read -r f - do - grep -e '[[:space:]]$' -n -- "$f"\ + find . \ + -type 'f' \ + '!' '(' \ + -name '*.db' \ + -o -name '*.exe' \ + -o -name '*.out' \ + -o -name '*.png' \ + -o -name '*.svg' \ + -o -name '*.tar.gz' \ + -o -name '*.test' \ + -o -name '*.zip' \ + -o -name 'AdGuardHome' \ + -o -name 'adguard-home' \ + -o -path '*/node_modules/*' \ + -o -path './.git/*' \ + -o -path './bin/*' \ + -o -path './build/*' \ + ')' \ + | while read -r f; do + grep -e '[[:space:]]$' -n -- "$f" \ | sed -e "s:^:${f}\::" -e 's/ \+$/>>>&<<&2 exit 1 @@ -128,41 +126,39 @@ in version="$last_tag" ;; -('candidate') +'candidate') # This pseudo-channel is used to set a proper versions into release # candidate builds. # last_tag is expected to be the latest release tag. - last_tag="$( git describe --abbrev=0 )" + last_tag="$(git describe --abbrev=0)" readonly last_tag # current_branch is the name of the branch currently checked out. - current_branch="$( git rev-parse --abbrev-ref HEAD )" + current_branch="$(git rev-parse --abbrev-ref HEAD)" readonly current_branch # The branch should be named like: # # rc-v12.34.56 # - if ! echo "$current_branch" | grep -E -e '^rc-v[0-9]+\.[0-9]+\.[0-9]+$' -q - then + if ! echo "$current_branch" | grep -E -e '^rc-v[0-9]+\.[0-9]+\.[0-9]+$' -q; then echo "invalid release candidate branch name '$current_branch'" 1>&2 exit 1 fi - version="${current_branch#rc-}-rc.$( git rev-list --count "$last_tag"..HEAD )" + version="${current_branch#rc-}-rc.$(git rev-list --count "$last_tag"..HEAD)" ;; -(*) - echo "invalid channel '$channel', supported values are\ +*) + echo "invalid channel '$channel', supported values are \ 'development', 'edge', 'beta', 'release' and 'candidate'" 1>&2 exit 1 ;; esac # Finally, make sure that we don't output invalid versions. -if ! echo "$version" | grep -E -e '^v[0-9]+\.[0-9]+\.[0-9]+(-(a|b|dev|rc)\.[0-9]+)?(\+[[:xdigit:]]+)?$' -q -then +if ! echo "$version" | grep -E -e '^v[0-9]+\.[0-9]+\.[0-9]+(-(a|b|dev|rc)\.[0-9]+)?(\+[[:xdigit:]]+)?$' -q; then echo "generated an invalid version '$version'" 1>&2 exit 1 diff --git a/scripts/snap/build.sh b/scripts/snap/build.sh index 53f487a1..083ae507 100644 --- a/scripts/snap/build.sh +++ b/scripts/snap/build.sh @@ -2,8 +2,7 @@ verbose="${VERBOSE:-0}" -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi @@ -14,16 +13,14 @@ set -e -f -u # # TODO(a.garipov): Add to helpers.sh and use more actively in scripts. log() { - if [ "$verbose" -gt '0' ] - then + if [ "$verbose" -gt '0' ]; then # Don't use quotes to get word splitting. echo "$1" 1>&2 fi } -version="$( ./AdGuardHome_amd64 --version | cut -d ' ' -f 4 )" -if [ "$version" = '' ] -then +version="$(./AdGuardHome_amd64 --version | cut -d ' ' -f 4)" +if [ "$version" = '' ]; then log 'empty version from ./AdGuardHome_amd64' exit 1 fi @@ -31,12 +28,11 @@ readonly version log "version '$version'" -for arch in\ - 'i386'\ - 'amd64'\ - 'armhf'\ - 'arm64' -do +for arch in \ + 'amd64' \ + 'arm64' \ + 'armhf' \ + 'i386'; do build_output="./AdGuardHome_${arch}" snap_output="./AdGuardHome_${arch}.snap" snap_dir="${snap_output}.dir" @@ -48,16 +44,16 @@ do cp -r './snap/gui' "${snap_dir}/meta/" # Create a snap.yaml file, setting the values. - sed\ - -e 's/%VERSION%/'"$version"'/'\ - -e 's/%ARCH%/'"$arch"'/'\ - ./snap/snap.tmpl.yaml\ - > "${snap_dir}/meta/snap.yaml" + sed \ + -e 's/%VERSION%/'"$version"'/' \ + -e 's/%ARCH%/'"$arch"'/' \ + ./snap/snap.tmpl.yaml \ + >"${snap_dir}/meta/snap.yaml" # TODO(a.garipov): The snapcraft tool will *always* write everything, - # including errors, to stdout. And there doesn't seem to be a way to change - # that. So, save the combined output, but only show it when snapcraft - # actually fails. + # including errors, to stdout. And there doesn't seem to be a way to + # change that. So, save the combined output, but only show it when + # snapcraft actually fails. set +e snapcraft_output="$( snapcraft pack "$snap_dir" --output "$snap_output" 2>&1 @@ -65,8 +61,7 @@ do snapcraft_exit_code="$?" set -e - if [ "$snapcraft_exit_code" -ne '0' ] - then + if [ "$snapcraft_exit_code" -ne '0' ]; then log "$snapcraft_output" exit "$snapcraft_exit_code" fi diff --git a/scripts/snap/download.sh b/scripts/snap/download.sh index 7eb03a0e..60ddc88f 100644 --- a/scripts/snap/download.sh +++ b/scripts/snap/download.sh @@ -2,8 +2,7 @@ verbose="${VERBOSE:-0}" -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi @@ -12,18 +11,17 @@ set -e -f -u channel="${CHANNEL:?please set CHANNEL}" readonly channel -printf '%s %s\n'\ - '386' 'i386'\ - 'amd64' 'amd64'\ - 'armv7' 'armhf'\ +printf '%s %s\n' \ + '386' 'i386' \ + 'amd64' 'amd64' \ + 'armv7' 'armhf' \ 'arm64' 'arm64' \ -| while read -r arch snap_arch -do - release_url="https://static.adtidy.org/adguardhome/${channel}/AdGuardHome_linux_${arch}.tar.gz" - output="./AdGuardHome_linux_${arch}.tar.gz" + | while read -r arch snap_arch; do + release_url="https://static.adtidy.org/adguardhome/${channel}/AdGuardHome_linux_${arch}.tar.gz" + output="./AdGuardHome_linux_${arch}.tar.gz" - curl -o "$output" -v "$release_url" - tar -f "$output" -v -x -z - cp ./AdGuardHome/AdGuardHome "./AdGuardHome_${snap_arch}" - rm -f -r "$output" ./AdGuardHome -done + curl -o "$output" -v "$release_url" + tar -f "$output" -v -x -z + cp ./AdGuardHome/AdGuardHome "./AdGuardHome_${snap_arch}" + rm -f -r "$output" ./AdGuardHome + done diff --git a/scripts/snap/upload.sh b/scripts/snap/upload.sh index e254ad7e..94a87e3c 100644 --- a/scripts/snap/upload.sh +++ b/scripts/snap/upload.sh @@ -2,8 +2,7 @@ verbose="${VERBOSE:-0}" -if [ "$verbose" -gt '0' ] -then +if [ "$verbose" -gt '0' ]; then set -x fi @@ -12,8 +11,7 @@ set -e -f -u # Function log is an echo wrapper that writes to stderr if the caller requested # verbosity level greater than 0. Otherwise, it does nothing. log() { - if [ "$verbose" -gt '0' ] - then + if [ "$verbose" -gt '0' ]; then # Don't use quotes to get word splitting. echo "$1" 1>&2 fi @@ -21,8 +19,7 @@ log() { # Do not set a new lowercase variable, because the snapcraft tool expects the # uppercase form. -if [ "${SNAPCRAFT_STORE_CREDENTIALS:-}" = '' ] -then +if [ "${SNAPCRAFT_STORE_CREDENTIALS:-}" = '' ]; then log 'please set SNAPCRAFT_STORE_CREDENTIALS' exit 1 @@ -40,12 +37,11 @@ default_timeout='90s' kill_timeout='120s' readonly default_timeout kill_timeout -for arch in\ - 'i386'\ - 'amd64'\ - 'armhf'\ - 'arm64' -do +for arch in \ + 'amd64' \ + 'arm64' \ + 'armhf' \ + 'i386'; do snap_file="./AdGuardHome_${arch}.snap" # Catch the exit code and the combined output to later inspect it. @@ -53,30 +49,28 @@ do snapcraft_output="$( # Use timeout(1) to force snapcraft to quit after a certain time. There # seems to be no environment variable or flag to force this behavior. - timeout\ - --preserve-status\ - -k "$kill_timeout"\ - -v "$default_timeout"\ - "$snapcraft_cmd" upload\ - --release="${snapcraft_channel}"\ - --quiet\ - "${snap_file}"\ + timeout \ + --preserve-status \ + -k "$kill_timeout" \ + -v "$default_timeout" \ + "$snapcraft_cmd" upload \ + --release="${snapcraft_channel}" \ + --quiet \ + "${snap_file}" \ 2>&1 )" snapcraft_exit_code="$?" set -e - if [ "$snapcraft_exit_code" -eq '0' ] - then + if [ "$snapcraft_exit_code" -eq '0' ]; then log "successful upload: ${snapcraft_output}" continue fi # Skip the ones that were failed by a duplicate upload error. - case "$snapcraft_output" - in - (*'A file with this exact same content has already been uploaded'*|\ + case "$snapcraft_output" in + *'A file with this exact same content has already been uploaded'* | \ *'Error checking upload uniqueness'*) log "warning: duplicate upload, skipping" @@ -84,7 +78,7 @@ do continue ;; - (*) + *) echo "unexpected snapcraft upload error: ${snapcraft_output}" return "$snapcraft_exit_code" diff --git a/scripts/translations/main.go b/scripts/translations/main.go index 33a57b19..c1f6a3cc 100644 --- a/scripts/translations/main.go +++ b/scripts/translations/main.go @@ -10,6 +10,7 @@ import ( "encoding/json" "fmt" "log/slog" + "maps" "net/url" "os" "os/exec" @@ -21,7 +22,6 @@ import ( "github.com/AdguardTeam/AdGuardHome/internal/aghos" "github.com/AdguardTeam/golibs/errors" "github.com/AdguardTeam/golibs/logutil/slogutil" - "golang.org/x/exp/maps" ) const ( @@ -76,40 +76,30 @@ func main() { usage("") } - conf, err := readTwoskyConfig() - check(err) + conf := errors.Must(readTwoskyConfig()) var cli *twoskyClient switch os.Args[1] { case "summary": - err = summary(conf.Languages) + errors.Check(summary(conf.Languages)) case "download": - cli, err = conf.toClient() - check(err) + cli = errors.Must(conf.toClient()) - err = cli.download(ctx, l) + errors.Check(cli.download(ctx, l)) case "unused": - err = unused(ctx, l, conf.LocalizableFiles[0]) + err := unused(ctx, l, conf.LocalizableFiles[0]) + errors.Check(err) case "upload": - cli, err = conf.toClient() - check(err) + cli = errors.Must(conf.toClient()) - err = cli.upload() + errors.Check(cli.upload()) case "auto-add": - err = autoAdd(conf.LocalizableFiles[0]) + err := autoAdd(conf.LocalizableFiles[0]) + errors.Check(err) default: usage("unknown command") } - - check(err) -} - -// check is a simple error-checking helper for scripts. -func check(err error) { - if err != nil { - panic(err) - } } // usage prints usage. If addStr is not empty print addStr and exit with code @@ -163,15 +153,11 @@ func readTwoskyConfig() (t *twoskyConfig, err error) { var tsc []twoskyConfig err = json.Unmarshal(b, &tsc) if err != nil { - err = fmt.Errorf("unmarshalling %q: %w", twoskyConfFile, err) - - return nil, err + return nil, fmt.Errorf("unmarshalling %q: %w", twoskyConfFile, err) } if len(tsc) == 0 { - err = fmt.Errorf("%q is empty", twoskyConfFile) - - return nil, err + return nil, fmt.Errorf("%q is empty", twoskyConfFile) } conf := tsc[0] @@ -224,7 +210,8 @@ func (t *twoskyConfig) toClient() (cli *twoskyClient, err error) { baseLang = langCode(uLangStr) } - langs := maps.Keys(t.Languages) + langs := slices.Sorted(maps.Keys(t.Languages)) + dlLangStr := os.Getenv("DOWNLOAD_LANGUAGES") if dlLangStr == "blocker" { langs = blockerLangCodes @@ -295,8 +282,7 @@ func summary(langs languages) (err error) { size := float64(len(baseLoc)) - keys := maps.Keys(langs) - slices.Sort(keys) + keys := slices.Sorted(maps.Keys(langs)) for _, lang := range keys { name := filepath.Join(localesDir, string(lang)+".json") @@ -399,10 +385,7 @@ func findUnused(fileNames []string, loc locales) (err error) { } } - keys := maps.Keys(loc) - slices.Sort(keys) - - for _, v := range keys { + for _, v := range slices.Sorted(maps.Keys(loc)) { fmt.Println(v) } diff --git a/scripts/vetted-filters/main.go b/scripts/vetted-filters/main.go index 4ccb29e5..c21e6837 100644 --- a/scripts/vetted-filters/main.go +++ b/scripts/vetted-filters/main.go @@ -13,6 +13,7 @@ import ( "os" "time" + "github.com/AdguardTeam/golibs/errors" "github.com/AdguardTeam/golibs/logutil/slogutil" "github.com/google/renameio/v2/maybe" ) @@ -22,20 +23,19 @@ func main() { l := slogutil.New(nil) urlStr := "https://adguardteam.github.io/HostlistsRegistry/assets/filters.json" - if v, ok := os.LookupEnv("URL"); ok { - urlStr = v + if s := os.Getenv("URL"); s != "" { + urlStr = s } // Validate the URL. _, err := url.Parse(urlStr) - check(err) + errors.Check(err) c := &http.Client{ Timeout: 10 * time.Second, } - resp, err := c.Get(urlStr) - check(err) + resp := errors.Must(c.Get(urlStr)) defer slogutil.CloseAndLog(ctx, l, resp.Body, slog.LevelError) if resp.StatusCode != http.StatusOK { @@ -44,7 +44,7 @@ func main() { hlFlt := &hlFilters{} err = json.NewDecoder(resp.Body).Decode(hlFlt) - check(err) + errors.Check(err) aghFlt := &aghFilters{ Categories: map[string]*aghFiltersCategory{ @@ -93,29 +93,21 @@ func main() { enc := json.NewEncoder(buf) enc.SetIndent("", " ") - err = enc.Encode(aghFlt) - check(err) + errors.Check(enc.Encode(aghFlt)) err = maybe.WriteFile("client/src/helpers/filters/filters.ts", buf.Bytes(), 0o644) - check(err) + errors.Check(err) } // jsHeader is the header for the generated JavaScript file. It informs the // reader that the file is generated and disables some style-related eslint -// checks. +// errors.Checks. const jsHeader = `// Code generated by go run ./scripts/vetted-filters/main.go; DO NOT EDIT. /* eslint quote-props: 'off', quotes: 'off', comma-dangle: 'off', semi: 'off' */ export default ` -// check is a simple error-checking helper for scripts. -func check(err error) { - if err != nil { - panic(err) - } -} - // hlFilters is the JSON structure for the Hostlists Registry rule list index. type hlFilters struct { Filters []*hlFiltersFilter `json:"filters"` diff --git a/snap/local/adguard-home-web.sh b/snap/local/adguard-home-web.sh index 3396dd17..46e9b966 100755 --- a/snap/local/adguard-home-web.sh +++ b/snap/local/adguard-home-web.sh @@ -1,24 +1,25 @@ #!/bin/sh +# shellcheck disable=SC2154 conf_file="${SNAP_DATA}/AdGuardHome.yaml" readonly conf_file -if ! [ -f "$conf_file" ] -then - xdg-open 'http://localhost:3000' +if ! [ -f "$conf_file" ]; then + xdg-open 'http://localhost:3000' - exit + exit fi # Get the admin interface port from the configuration. +# +# shellcheck disable=SC2016 awk_prog='/^[^[:space:]]/ { is_http = /^http:/ };/^[[:space:]]+address:/ { if (is_http) print $2 }' readonly awk_prog -bind_port="$( awk "$awk_prog" "$conf_file" | awk -F ':' '{print $NF}' )" +bind_port="$(awk "$awk_prog" "$conf_file" | awk -F ':' '{print $NF}')" readonly bind_port -if [ "$bind_port" = '' ] -then +if [ "$bind_port" = '' ]; then xdg-open 'http://localhost:3000' else xdg-open "http://localhost:${bind_port}"