Linters and SAST

feat: modify rpc methods to include synthetic txs #4336

Sign in to view logs

Triggered via pull request May 28, 2024 22:19

skosito

synchronize #2282

debug-trace-tx-changes

Status Failure

Total duration 4m 27s

Artifacts –

sast-linters.yml

on: pull_request

Annotations

6 errors and 3 warnings

gosec-cosmos

Process completed with exit code 2.

lint: rpc/backend/tracing.go#L57

Consider pre-allocating `predecessors` (prealloc)

lint: rpc/types/events.go#L380

unnecessary conversion (unconvert)

lint: rpc/types/events.go#L389

unnecessary conversion (unconvert)

lint: rpc/types/events.go#L403

unnecessary conversion (unconvert)

issues found

nosec_alert

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/github-script@v6. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

lint: rpc/backend/blocks.go#L319

superfluous-else: if block ends with a continue statement, so drop this else and outdent its block (revive)