Closed insumity closed 4 months ago
x/ccv/provider/keeper/proposal.go (1)
Pattern `**/*.go`: Review the Golang code for conformity with the Uber Golang style guide, highlighting any deviations.
.changelog/unreleased/dependencies/1924-bump-ibc.md
3-3: null (MD047, single-trailing-newline) Files should end with a single newline character
Description
Closes (partially): https://github.com/cosmos/interchain-security/issues/1926
Attached Audit Report
This PR applies some of the suggestions suggested by the audit team in regards to PSS. The change in
NoMoreThanPercentOfTheSum
on how we computemaxPower
might have as a result, different powers being sent down to a consumer chain. Therefore, this is a breaking change and I'm not backporting it tov4.2.x
orv4.2.x-lsm
.This PR also introduces new checks on the returner
err
or fromComputeMinPowerToOptIn
. Thanks to @sainoe for pointing this out!Author Checklist
All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.
I have...
!
to the type prefix if the change is state-machine breakingCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.
I have...
!
the type prefix if the change is state-machine breakingSummary by CodeRabbit
Refactor
Bug Fixes
Tests