Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: write channel state before invoking app callbacks in ack and confirm channel handshake steps #2973

Merged
merged 5 commits into from
Jan 18, 2023
Merged

chore: write channel state before invoking app callbacks in ack and confirm channel handshake steps #2973

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f666e95
write channel state before app cbs in ack and confirm channel handshakes
damiannolan Dec 23, 2022
1b61b2c
Merge branch 'main' into damian/2962-persist-channel-state-before-app…
damiannolan Jan 6, 2023
a49abbc
Merge branch 'main' into damian/2962-persist-channel-state-before-app…
damiannolan Jan 11, 2023
a58dd78
Merge branch 'main' into damian/2962-persist-channel-state-before-app…
damiannolan Jan 16, 2023
b929426
Merge branch 'main' into damian/2962-persist-channel-state-before-app…
crodriguezvega Jan 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into damian/2962-persist-channel-state-before-app… 
…-cbs
  • Loading branch information
@damiannolan
damiannolan authored Jan 11, 2023
commit a49abbc0b79a16d692eac44244b0893fd67d01dd
4 changes: 4 additions & 0 deletions modules/core/keeper/msg_server.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -291,6 +291,8 @@ func (k Keeper) ChannelOpenAck(goCtx context.Context, msg *channeltypes.MsgChann
return nil, sdkerrors.Wrapf(err, "channel open ack callback failed for port ID: %s, channel ID: %s", msg.PortId, msg.ChannelId)
}

ctx.Logger().Info("channel open ack callback succeeded", "channel-id", msg.ChannelId, "port-id", msg.PortId)

return &channeltypes.MsgChannelOpenAckResponse{}, nil
}

Expand Down Expand Up @@ -329,6 +331,8 @@ func (k Keeper) ChannelOpenConfirm(goCtx context.Context, msg *channeltypes.MsgC
return nil, sdkerrors.Wrapf(err, "channel open confirm callback failed for port ID: %s, channel ID: %s", msg.PortId, msg.ChannelId)
}

ctx.Logger().Info("channel open confirm callback succeeded", "channel-id", msg.ChannelId, "port-id", msg.PortId)

return &channeltypes.MsgChannelOpenConfirmResponse{}, nil
}

Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.