Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WILL NOT BE MERGED] Kava v0.50.10 patch #84

Closed
wants to merge 81 commits into from

wip: add logging to debug traces

b04d78f
Select commit
Loading
Failed to load commit list.
Closed

[WILL NOT BE MERGED] Kava v0.50.10 patch #84

wip: add logging to debug traces
b04d78f
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded Nov 14, 2024 in 8s

4 new alerts

New alerts in code changed by this pull request

  • 4 warnings

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 154 in app/app.go

See this annotation in the file changed.

Code scanning / CodeQL

Directly using the bech32 constants Warning

Directly using the bech32 constants instead of the configuration values

Check warning on line 619 in server/start.go

See this annotation in the file changed.

Code scanning / CodeQL

Useless assignment to local variable Warning

This definition of clientCtx is never used.

Check warning on line 370 in x/evm/keeper/grpc_query.go

See this annotation in the file changed.

Code scanning / CodeQL

Missing error check Warning

rsp
Loading
may be nil at this dereference because
err
Loading
may not have been checked.

Check warning on line 168 in x/evm/module.go

See this annotation in the file changed.

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning

path flow from Begin/EndBlock to a panic call
path flow from Begin/EndBlock to a panic call