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

Panic in WriteErrorReceipt instead of returning an error #5207

Closed
Tracked by #5201
crodriguezvega opened this issue Nov 29, 2023 · 1 comment
Closed
Tracked by #5201

Panic in WriteErrorReceipt instead of returning an error #5207

crodriguezvega opened this issue Nov 29, 2023 · 1 comment
Assignees
@hieuvubk

Comments

@crodriguezvega
Copy link
Contributor

crodriguezvega commented Nov 29, 2023 •
edited
Loading

Panic in WriteErrorReceipt instead of returning an error, which is ignored.
@crodriguezvega crodriguezvega mentioned this issue Nov 29, 2023
Closed
@crodriguezvega crodriguezvega changed the title Panic in WriteErrorReceipt [instead of returning an error](https://github.com/cosmos/ibc-go/blob/39df8d2b579fc623a1e748b4330738ff40adb682/modules/core/04-channel/keeper/upgrade.go#L932), which is ignored. Panic in WriteErrorReceipt instead of returning an error Nov 29, 2023
@hieuvubk
Copy link
Contributor

I will take this one, thanks @crodriguezvega

@chatton chatton mentioned this issue Nov 29, 2023
Merged
9 tasks
@hieuvubk hieuvubk mentioned this issue Nov 29, 2023
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hieuvubk hieuvubk

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@damiannolan @crodriguezvega @hieuvubk