ECR registry using IAM role in Kubernetes (#2307) #2348
ramanan-raviAnnotations
21 errors and 4 warnings
|
lint-bootstrapper:
deepfence_bootstrapper/main.go#L133
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint-bootstrapper:
deepfence_bootstrapper/main.go#L149
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint-bootstrapper:
deepfence_bootstrapper/main.go#L164
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint-bootstrapper:
deepfence_bootstrapper/main.go#L49
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
lint-bootstrapper:
deepfence_bootstrapper/main.go#L39
var `binaryOnly` is unused (unused)
|
|
lint-bootstrapper:
deepfence_bootstrapper/router/cloud_scanner.go#L181
Error return value of `conn.SetReadDeadline` is not checked (errcheck)
|
|
lint-bootstrapper:
deepfence_bootstrapper/router/cloud_scanner.go#L182
unslice: could simplify buf[:] to buf (gocritic)
|
|
lint-bootstrapper:
deepfence_bootstrapper/router/upgrade.go#L82
exitAfterDefer: log.Fatal will exit, and `defer os.Remove(dir)` will not run (gocritic)
|
|
lint-bootstrapper:
deepfence_bootstrapper/router/cloud_scanner.go#L192
SA4004: the surrounding loop is unconditionally terminated (staticcheck)
|
|
lint-bootstrapper
issues found
|
|
lint-utils
golangci-lint exit with code 3
|
|
lint-server:
deepfence_server/pkg/constants/version.go#L17
S1025: the argument is already a string, there's no need to use fmt.Sprintf (gosimple)
|
|
lint-server:
deepfence_server/pkg/registry/gitlab/client.go#L111
File is not `gofmt`-ed with `-s` (gofmt)
|
|
lint-server:
deepfence_server/pkg/registry/gitlab/client.go#L112
S1000: should use a simple channel send/receive instead of `select` with a single case (gosimple)
|
|
lint-server:
deepfence_server/pkg/registry/dockerprivate/client.go#L69
File is not `gofmt`-ed with `-s` (gofmt)
|
|
lint-server:
deepfence_server/pkg/registry/dockerprivate/client.go#L70
S1000: should use a simple channel send/receive instead of `select` with a single case (gosimple)
|
|
lint-server:
deepfence_server/pkg/registry/acr/client.go#L61
File is not `gofmt`-ed with `-s` (gofmt)
|
|
lint-server:
deepfence_server/pkg/registry/acr/client.go#L62
S1000: should use a simple channel send/receive instead of `select` with a single case (gosimple)
|
|
lint-server:
deepfence_server/apiDocs/operation.go#L654
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint-server:
deepfence_server/pkg/registry/ecr/client.go#L58
func `getAWSAccountID` is unused (unused)
|
|
lint-server:
deepfence_server/pkg/registry/ecr/ecr.go#L17
var `errAccountIDMissing` is unused (unused)
|
|
lint-bootstrapper
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint-utils
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint-server
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
lint-server
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|