Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

If a query panics an error acknowledgement is not sent. #64

Closed
VitalyV1337 opened this issue Aug 14, 2023 · 1 comment
Closed

If a query panics an error acknowledgement is not sent. #64

VitalyV1337 opened this issue Aug 14, 2023 · 1 comment

Comments

@VitalyV1337
Copy link

Example:

Let's say I want to query GetCmdEstimateSwapExactAmountIn from Osmosis, if you're querying a stableswap pool and enter more tokens than there are in a pool it will panic Here this will not cause the query to return an error thus async-icq will not return an ack-err.
@akc2267
Copy link

akc2267 commented Aug 14, 2023

Hey please follow up with Osmosis team in the Cosmos/ibc-apps repo

@akc2267 akc2267 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@akc2267 @VitalyV1337