keep default action as reject and make all error actions explicit
This commit is contained in:
parent
be3281bdcf
commit
bc5637c60a
@ -350,16 +350,32 @@ func (mv *MessageValidator) Validate(ctx context.Context, pid peer.ID, msg *pubs
|
|||||||
)
|
)
|
||||||
recordFailure(ctx, metrics.MessageValidationFailure, "add")
|
recordFailure(ctx, metrics.MessageValidationFailure, "add")
|
||||||
switch {
|
switch {
|
||||||
|
|
||||||
|
case xerrors.Is(err, messagepool.ErrSoftValidationFailure):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrRBFTooLowPremium):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrTooManyPendingMessages):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrNonceGap):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrGasFeeCapTooLow):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrNonceTooLow):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrNotEnoughFunds):
|
||||||
|
fallthrough
|
||||||
|
case xerrors.Is(err, messagepool.ErrExistingNonce):
|
||||||
|
return pubsub.ValidationIgnore
|
||||||
|
|
||||||
case xerrors.Is(err, messagepool.ErrMessageTooBig):
|
case xerrors.Is(err, messagepool.ErrMessageTooBig):
|
||||||
fallthrough
|
fallthrough
|
||||||
case xerrors.Is(err, messagepool.ErrMessageValueTooHigh):
|
case xerrors.Is(err, messagepool.ErrMessageValueTooHigh):
|
||||||
fallthrough
|
fallthrough
|
||||||
case xerrors.Is(err, messagepool.ErrNotEnoughFunds):
|
|
||||||
fallthrough
|
|
||||||
case xerrors.Is(err, messagepool.ErrInvalidToAddr):
|
case xerrors.Is(err, messagepool.ErrInvalidToAddr):
|
||||||
return pubsub.ValidationReject
|
fallthrough
|
||||||
default:
|
default:
|
||||||
return pubsub.ValidationIgnore
|
return pubsub.ValidationReject
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user