Move handleAbort to msg server layer

[colin-axner]

Summary

see discussion, move handle abort to msg server layer so it can call the on restore callback

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned

[damiannolan]

I think we can possibly leave the MustAbortUpgrade function in 04-channel now? And just add a simple callback to the app within the core/keeper handler:

if upgradeErr, ok := err.(*channeltypes.UpgradeError); ok {
	k.ChannelKeeper.MustAbortUpgrade(ctx, msg.PortId, msg.ChannelId, upgradeErr)

+	cbs.OnChanUpgradeRestore(ctx, msg.PortId, msg.ChannelId)

	// NOTE: a FAILURE result is returned to the client and an error receipt is written to state.
	// This signals to the relayer to begin the cancel upgrade handshake subprotocol.
	return &channeltypes.MsgChannelUpgradeAckResponse{Result: channeltypes.FAILURE}, nil
}

[DimitrisJim]

bump, issue can be closed yea?

[damiannolan]

Yeah, I opened #4056 linking to the discussion in PR #4051