Skip to content

Commit

Permalink
go/consensus/tendermint/apps/keymanager: Remove secure transition check
Browse files Browse the repository at this point in the history
Removing legacy code which assumed that key managers will be set in
the genesis file before being initialized.
  • Loading branch information
peternose committed Feb 23, 2023
1 parent eaff1b8 commit 61b37fc
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
Empty file added .changelog/5197.trivial.md
Empty file.
7 changes: 0 additions & 7 deletions go/consensus/tendermint/apps/keymanager/keymanager.go
Original file line number Diff line number Diff line change
Expand Up @@ -304,13 +304,6 @@ nextNode:
// Set immutable status fields that cannot change after initialization.
if !isInitialized {
// The first version gets to be the source of truth.

// Allow false -> true transitions, but not the reverse, so that
// it is possible to set the security status in the genesis block.
if initResponse.IsSecure != isSecure && !initResponse.IsSecure {
ctx.Logger().Error("Security status mismatch for runtime", vars...)
continue nextNode
}
isInitialized = true
isSecure = initResponse.IsSecure
checksum = initResponse.Checksum
Expand Down

0 comments on commit 61b37fc

Please sign in to comment.