Skip to content

ipn/proxies: pin or return at least one err #982

ipn/proxies: pin or return at least one err

ipn/proxies: pin or return at least one err #982

Triggered via push November 24, 2024 21:18
Status Failure
Total duration 5m 13s
Artifacts

go.yml

on: push
🧬 Build
5m 4s
🧬 Build
🔐 Security checker
28s
🔐 Security checker
🧭 Lint
34s
🧭 Lint
Fit to window
Zoom out
Zoom in

Annotations

23 errors, 11 warnings, and 1 notice
🧭 Lint: intra/ipn/auto.go#L350
func baID is unused (U1000)
🧭 Lint: intra/ipn/proxy.go#L284
func icmpReachesWork is unused (U1000)
🧭 Lint
Process completed with exit code 1.
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧭 Lint
Cannot open: File exists
🧬 Build: intra/dns53/dot.go#L293
error: Potential nil panic detected. Observed nil flow from source to dereference point:
🧬 Build
result 0 of `pinID()` lacking guarding; returned from `ProxyTo()` in position 0
🧬 Build
returned as result 0 from interface method `Proxies.ProxyTo()` (implemented by `proxifier.ProxyTo()`)
🧬 Build
result 0 of `ProxyTo()` called `ID()` via the assignment(s):
🧬 Build: intra/dnscrypt/servers.go#L444
error: Potential nil panic detected. Observed nil flow from source to dereference point:
🧬 Build
result 0 of `pinID()` lacking guarding; returned from `ProxyTo()` in position 0
🧬 Build
returned as result 0 from interface method `Proxies.ProxyTo()` (implemented by `proxifier.ProxyTo()`)
🧬 Build
result 0 of `ProxyTo()` called `ID()` via the assignment(s):
🧬 Build: intra/doh/doh.go#L688
error: Potential nil panic detected. Observed nil flow from source to dereference point:
🧬 Build
result 0 of `pinID()` lacking guarding; returned from `ProxyTo()` in position 0
🧭 Lint
The following actions use a deprecated Node.js version and will be forced to run on node20: 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
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
🧭 Lint: intra/backend/core_iptree.go#L7
package-comments: should have a package comment (revive)
🧭 Lint: intra/backend/core_iptree.go#L22
var-naming: type IpTree should be IPTree (revive)
🧭 Lint: intra/backend/core_iptree.go#L67
exported: exported const Vsep should have comment (or a comment on this block) or be unexported (revive)
🧭 Lint: intra/backend/core_iptree.go#L78
var-naming: func NewIpTree should be NewIPTree (revive)
🧭 Lint: intra/backend/core_iptree.go#L97
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#L147
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#L232
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#L280
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
🧭 Lint: intra/backend/core_iptree.go#L299
unused-parameter: parameter 'k' seems to be unused, consider removing or renaming it as _ (revive)
🧬 Build
success