cosmos / ibc-apps

IBC applications and middleware for Cosmos SDK chains.
Apache License 2.0
82 stars 64 forks source link

`[BP: release/v7 <- #216]` refactor: remove processed key and handling code #218

Closed mergify[bot] closed 2 months ago

mergify[bot] commented 2 months ago

This PR removes the ProcessedKey, which was previously being used by the swap and forward middleware to determine if a packet being handled was already processed by a middleware further up in the stack. If a packet was previously processed by some other middleware then this value would be initialized via a value passed through a context, this would ensure the PFM would not make a second call into the underlying app and invoke the OnRecvPacket callback twice, ensuring that the mint/burn voucher logic would not be invoked more than once.

If for some reason there is a need to have this sort of handling in the future then we can revisit bringing in this code.


This is an automatic backport of pull request #216 done by Mergify.