Skip to content

ipn/wg: s/loged/logev #899

ipn/wg: s/loged/logev

ipn/wg: s/loged/logev #899

Triggered via push October 25, 2024 12:31
Status Failure
Total duration 4m 54s
Artifacts

go.yml

on: push
🧬 Build
4m 44s
🔐 Security checker
25s
🧭 Lint
22s

Annotations

18 errors, 12 warnings, and 1 notice
🧭 Lint: intra/common.go#L595
func firstEmpty is unused (U1000)
🧭 Lint: intra/udp.go#L53
var errNoProxyID is unused (U1000)
🧭 Lint
Process completed with exit code 1.
🧭 Lint: intra/bootstrap.go#L179
Error return value of `prev.Stop` is not checked (errcheck)
🧭 Lint: intra/common.go#L595
func `firstEmpty` is unused (unused)
🧭 Lint: intra/dialers/direct_split.go#L31
`strat` is a misspelling of `start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L155
`strat` is a misspelling of `start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L156
`Strat` is a misspelling of `Start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L158
`Strat` is a misspelling of `Start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L178
`strat` is a misspelling of `start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L184
`strat` is a misspelling of `start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L188
`strat` is a misspelling of `start` (misspell)
🧭 Lint: intra/dialers/retrier.go#L190
`strat` is a misspelling of `start` (misspell)
🧬 Build: intra/udp.go#L327
error: Potential nil panic detected. Observed nil flow from source to dereference point:
🧬 Build
result 0 of `ProxyTo()` lacking guarding; called `ID()` via the assignment(s):
🧬 Build: intra/udp.go#L327
error: Potential nil panic detected. Observed nil flow from source to dereference point:
🧬 Build
unassigned variable `px` called `ID()`
🧬 Build
Process completed with exit code 3.
🧭 Lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4, golangci/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
🧭 Lint: intra/backend/core_iptree.go#L7
package-comments: should have a package comment (revive)
🧭 Lint: intra/backend/core_iptree.go#L21
var-naming: type IpTree should be IPTree (revive)
🧭 Lint: intra/backend/core_iptree.go#L77
var-naming: func NewIpTree should be NewIPTree (revive)
🧭 Lint: intra/backend/core_iptree.go#L96
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
🧭 Lint: intra/backend/core_iptree.go#L146
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
🧭 Lint: intra/backend/core_iptree.go#L231
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
🧭 Lint: intra/backend/core_iptree.go#L279
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
🧭 Lint: intra/backend/core_iptree.go#L298
unused-parameter: parameter 'k' seems to be unused, consider removing or renaming it as _ (revive)
🧭 Lint: intra/backend/core_iptree.go#L339
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
🧭 Lint: intra/backend/core_iptree.go#L361
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
🧬 Build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
🧬 Build
success