IBC event for channel_open_init has module ibc_connection

[ancazamfir]

Summary of Bug

IBC event for channel_open_init has module ibc_connection instead of ibc_channel

Version

Tested hermes with gaia v6.0.0:

	github.com/cosmos/cosmos-sdk v0.44.3
	github.com/cosmos/ibc-go/v2 v2.0.0

With gaia v5.0.8 the module is ibc_channel.

Steps to Reproduce

Start two gaia chains, create clients and connection, then hermes tx raw chan-open-init ibc-0 ibc-1 connection-0 transfer transfer while listening with hermes listen ibc-0 -e Tx. I can provide more details if required.
A custom hermes trace shows:

            Event {
                type_str: "channel_open_init",
                attributes: [
                    Tag {
                        key: Key(
                            "port_id",
                        ),
                        value: Value(
                            "transfer",
                        ),
                    },
                    Tag {
                        key: Key(
                            "channel_id",
                        ),
                        value: Value(
                            "channel-1",
                        ),
                    },
                    Tag {
                        key: Key(
                            "counterparty_port_id",
                        ),
                        value: Value(
                            "transfer",
                        ),
                    },
                    Tag {
                        key: Key(
                            "counterparty_channel_id",
                        ),
                        value: Value(
                            "",
                        ),
                    },
                    Tag {
                        key: Key(
                            "connection_id",
                        ),
                        value: Value(
                            "connection-0",
                        ),
                    },
                ],
            },
            Event {
                type_str: "message",
                attributes: [
                    Tag {
                        key: Key(
                            "module",
                        ),
                        value: Value(
                            "ibc_connection",
                        ),
                    },
                ],
            },

---

For Admin Use

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

[seantking]

Interesting! I see that we are emitting the correct event on our side here

I'll take a look at the logs on-chain rather than the hermes side.

[seantking]

Ah actually we are emitting the module part here. I see we are using connectiontypes

[crodriguezvega]

Thank you for this finding, @ancazamfir!

[colin-axner]

nice catch @ancazamfir! Thank you!