Closed ws4charlie closed 4 days ago
[!IMPORTANT]
Review skipped
Auto incremental reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 69.73684%
with 23 lines
in your changes missing coverage. Please review.
Project coverage is 67.27%. Comparing base (
d14b200
) to head (49f3fef
). Report is 1 commits behind head on develop.
Files with missing lines | Patch % | Lines |
---|---|---|
zetaclient/chains/bitcoin/observer/inbound.go | 46.51% | 23 Missing :warning: |
!!!WARNING!!!
nosec
detected in the following files: zetaclient/chains/bitcoin/observer/inbound.go
Be very careful about using #nosec
in code. It can be a quick way to suppress security warnings and move forward with development, it should be employed with caution. Suppressing warnings with #nosec can hide potentially serious vulnerabilities. Only use #nosec when you're absolutely certain that the security issue is either a false positive or has been mitigated in another way.
Only suppress a single rule (or a specific set of rules) within a section of code, while continuing to scan for other problems. To do this, you can list the rule(s) to be suppressed within the #nosec annotation, e.g: / #nosec G401 / or //#nosec G201 G202 G203
Broad #nosec
annotations should be avoided, as they can hide other vulnerabilities. The CI will block you from merging this PR until you remove #nosec
annotations that do not target specific rules.
Pay extra attention to the way #nosec
is being used in the files listed above.
Description
CalcDepositorFeeV1
IsBlockConfirmed
; BothWatchInbound
andWatchInboundTracker
tickers use it.Closes: https://github.com/zeta-chain/node/issues/2766
How Has This Been Tested?
Summary by CodeRabbit
New Features
Refactor
Bug Fixes
Documentation