Set active channel update

[seantking]

Summary

On the controller side, we should set the active channel on the OnChanOpenAck step. On the host side, we should set the active channel on the OnChanOpenConfirm step.

Additionally, OnChanCloseConfirm should delete the current active channel mapping.

• Set active channel correctly
• Delete the active channel on channel close

---

For Admin Use

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

[damiannolan]

Additionally, OnChanCloseConfirm should delete the current active channel mapping.

This will only delete the active channel mapping for one party. I think we'll have to also delete the active channel mapping in OnTimeoutPacket. So the scenario there would be:

1. Controller calls TrySendTx
2. Packet times out.
3. Relayer calls Timeout gRPC handler on Controller, which will in turn closes the channel because it's ORDERED.
4. Relayer calls ChannelCloseConfirm grpc handler on Host

[colin-axner]

Yes, that sounds correct to me

[seantking]

Good catch. Agree, we need to add this! I'm wondering if there are any race conditions we need to consider here, especially now that we are allowing for timeouts again.

[damiannolan]

I'll include this in #463 🤝