doh: err on nil extra rr; avoid nil rr headers #840
ignoramousAnnotations
13 errors, 12 warnings, and 1 notice
|
🧭 Lint:
intra/dialers/direct_split.go#L31
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/direct_split.go#L38
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/direct_split.go#L56
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/direct_split.go#L57
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L144
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L145
`Strat` is a misspelling of `Start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L147
`Strat` is a misspelling of `Start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L167
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L173
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L177
`strat` is a misspelling of `start` (misspell)
|
|
🧬 Build:
intra/doh/doh.go#L354
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
|
🧬 Build
result 0 of `AddEdnsPadding()` lacking guarding; accessed field `Id` via the assignment(s):
|
|
🧬 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
|