search

sei-protocol / sei-tendermint

Tendermint fork with ABCI++ support, bug fixes, and custom features
Other
39 stars 36 forks source link

Remove tx from cache when canAddPendingTx fails #230

Closed yzang2019 closed 7 months ago

yzang2019 commented 7 months ago

Describe your changes and provide context

When PendingTxs is full, canAddPendingTx fails here, and the removeHandler doesn’t execute. This behavior prevent the addition of identical transactions, as an tx already exists in cache error occurs here.

Testing performed to validate your change

Added unit test coverage