intra/tunnel: serialize access to tunnel.Tunnel to avoid race #1265
ignoramousAnnotations
30 errors, 11 warnings, and 1 notice
|
🧭 Lint:
intra/ipn/warp/yegor.go#L122
var errWsSessionExpired is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L101
const wsbestloc is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L95
const wspxpath is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L40
const wsMyIp2 is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L39
const wsMyIp is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L37
const wsProdAssets2 is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L36
const wsProdAssets is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/yegor.go#L35
const wsTestAssets is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/regional.go#L127
func (*RegionalWgConf).genWgConf is unused (U1000)
|
|
🧭 Lint:
intra/ipn/warp/regional.go#L102
func (*RegionalWgConf).dnsCsv is unused (U1000)
|
|
🧭 Lint:
intra/dialers/retrier.go#L276
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L274
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L270
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L264
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L244
`Strat` is a misspelling of `Start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L242
`Strat` is a misspelling of `Start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L241
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/dialers/retrier.go#L168
`Strat` is a misspelling of `Start` (misspell)
|
|
🧭 Lint:
intra/dialers/direct_split.go#L31
`strat` is a misspelling of `start` (misspell)
|
|
🧭 Lint:
intra/core/connpool.go#L443
Error return value of `a.c.SetDeadline` is not checked (errcheck)
|
|
🧬 Build
literal `nil` returned from `config()` in position 0
|
|
🧬 Build:
intra/ipn/warp/yegor.go#L1454
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
|
🧬 Build
result 0 of `wsErr2()` lacking guarding; accessed field `Code` via the assignment(s):
|
|
🧬 Build:
intra/ipn/warp/yegor.go#L1292
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
|
🧬 Build
result 0 of `wsErr2()` lacking guarding; accessed field `Code` via the assignment(s):
|
|
🧬 Build:
intra/ipn/warp/yegor.go#L1219
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
|
🧬 Build
result 0 of `Do()` accessed field `StatusCode` via the assignment(s):
|
|
🧬 Build
result 0 of `do()` returned from `Do()` in position 0
|
|
🧬 Build
unassigned variable `resp` returned from `do()` in position 0
|
|
🧬 Build:
intra/ipn/warp/yegor.go#L966
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
|
🧭 Lint
Failed to save: <h2>Our services aren't available right now</h2><p>We're working to restore all services as soon as possible. Please check back soon.</p>0Kbd/aAAAAAAGibjL5ECKT7rkz/9erm4+QkwyQUEyMDMwMTA1MDQ1AEVkZ2U=
|
|
🧭 Lint:
intra/backend/core_boxes.go#L129
redefines-builtin-id: redefinition of the built-in function new (revive)
|
|
🧭 Lint:
intra/backend/core_boxes.go#L115
exported: exported function StrOf2 should have comment or be unexported (revive)
|
|
🧭 Lint:
intra/backend/core_boxes.go#L111
exported: exported function StrOf should have comment or be unexported (revive)
|
|
🧭 Lint:
intra/backend/core_boxes.go#L104
receiver-naming: receiver name st should be consistent with previous receiver name s for Gostr (revive)
|
|
🧭 Lint:
intra/backend/core_boxes.go#L97
exported: exported method Gostr.V should have comment or be unexported (revive)
|
|
🧭 Lint:
intra/backend/core_boxes.go#L59
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_boxes.go#L52
increment-decrement: should replace tries += 1 with tries++ (revive)
|
|
🧭 Lint:
intra/backend/core_boxes.go#L35
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_boxes.go#L7
package-comments: should have a package comment (revive)
|
|
🧭 Lint
Failed to restore: Cache service responded with 400
|
|
🧬 Build
success
|