refactor: remove processed key and handling code

middleware/packet-forward-middleware/packetforward/ibc_middleware.go

@@ -196,7 +196,6 @@ func (im IBCMiddleware) OnRecvPacket(
 	metadata := m.Forward
 
 	goCtx := ctx.Context()
-	processed := getBoolFromAny(goCtx.Value(types.ProcessedKey{}))
 	nonrefundable := getBoolFromAny(goCtx.Value(types.NonrefundableKey{}))
 	disableDenomComposition := getBoolFromAny(goCtx.Value(types.DisableDenomCompositionKey{}))
 
@@ -212,16 +211,6 @@ func (im IBCMiddleware) OnRecvPacket(
 		return newErrorAcknowledgement(fmt.Errorf("failed to construct override receiver: %w", err))
 	}
 
-	// if this packet has been handled by another middleware in the stack there may be no need to call into the
-	// underlying app, otherwise the transfer module's OnRecvPacket callback could be invoked more than once
-	// which would mint/burn vouchers more than once
-	if !processed {
-		if err := im.receiveFunds(ctx, packet, data, overrideReceiver, relayer); err != nil {
-			logger.Error("packetForwardMiddleware OnRecvPacket error receiving packet", "error", err)
-			return newErrorAcknowledgement(fmt.Errorf("error receiving packet: %w", err))
-		}
-	}
-
 	// if this packet's token denom is already the base denom for some native token on this chain,
 	// we do not need to do any further composition of the denom before forwarding the packet
 	denomOnThisChain := data.Denom

middleware/packet-forward-middleware/packetforward/types/keys.go

@@ -13,12 +13,9 @@ const (
 	QuerierRoute = ModuleName
 )
 
-var ParamsKey = []byte{0x00}
-
 type (
 	NonrefundableKey           struct{}
 	DisableDenomCompositionKey struct{}
-	ProcessedKey               struct{}
 )
 
 func RefundPacketKey(channelID, portID string, sequence uint64) []byte {