ChronosXYZ/dione
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add warning when we don't have payload validation for specific request

Browse Source
This commit is contained in:
ChronosX88 2020-12-03 01:21:10 +04:00
parent d6ab3503bc
commit e3ce7f4175
Signed by: ChronosXYZ
GPG Key ID: 085A69A82C8C511A
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
consensus/pre_prepare_pool.go
View File

@ -4,8 +4,6 @@ import (
"fmt" "fmt"
"github.com/Secured-Finance/dione/consensus/validation" "github.com/Secured-Finance/dione/consensus/validation"
rtypes "github.com/Secured-Finance/dione/rpc/types"
oracleEmitter "github.com/Secured-Finance/dione/contracts/oracleemitter" oracleEmitter "github.com/Secured-Finance/dione/contracts/oracleemitter"
"github.com/Secured-Finance/dione/node" "github.com/Secured-Finance/dione/node"
@ -166,11 +164,13 @@ func (ppp *PrePreparePool) IsValidPrePrepare(prePrepare *types2.Message) bool {
////////////////////////////////////// //////////////////////////////////////
// === validate payload by specific-chain checks === // === validate payload by specific-chain checks ===
if validationFunc := validation.GetValidationMethod(rtypes.RPCTypeFilecoin, consensusMsg.Task.RequestType); validationFunc != nil { if validationFunc := validation.GetValidationMethod(consensusMsg.Task.OriginChain, consensusMsg.Task.RequestType); validationFunc != nil {
err := validationFunc(consensusMsg.Task.Payload) err := validationFunc(consensusMsg.Task.Payload)
if err != nil { if err != nil {
logrus.Errorf("payload validation has failed: %v", err) logrus.Errorf("payload validation has failed: %v", err)
} }
} else {
logrus.Warnf("Origin chain [%v]/request type[%v] doesn't have any payload validation!", consensusMsg.Task.OriginChain, consensusMsg.Task.RequestType)
} }
///////////////////////////////// /////////////////////////////////