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.

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

poolmanager::EstimateSwapExactAmountOut failed with no sender field #21

Closed
byeongsu-hong opened this issue May 3, 2023 · 1 comment
Closed

Comments

@byeongsu-hong
Copy link

byeongsu-hong commented May 3, 2023

I was using autobuild-15.0.0 to test an IBCX contract. but after bump to v15.1.0, this situation happened.

before bump

expected(raw)         : 7b22746f6b656e5f696e5f616d6f756e74223a2231303030303030227d
expected(hex_decoded) : {"token_in_amount":"1000000"}

actual(raw)           : 7b227374617267617465223a7b2270617468223a222f6f736d6f7369732e706f6f6c6d616e616765722e763162657461312e51756572792f457374696d617465537761704578616374416d6f756e744f7574222c2264617461223a22436763784d4441774d444177227d7d
actual(hex_decoded)   : {"stargate":{"path":"/osmosis.poolmanager.v1beta1.Query/EstimateSwapExactAmountOut","data":"CgcxMDAwMDAw"}}

There's also a bug that simulation result is returned as Request type - osmosis-labs/osmosis#5048 - so i solved with decoding both type of response EstimateSwapExactAmountOutRequest and EstimateSwapExactAmountOutResponse and cherrypick the successful one.

But after bump it returns without totally different format like this.

expected(raw)         : 7b22746f6b656e5f696e5f616d6f756e74223a2231303030303030227d
expected(hex_decoded) : {"token_in_amount":"1000000"}

actual(raw)           : 7b22706f6f6c5f6964223a2230222c22726f75746573223a5b5d2c22746f6b656e5f6f7574223a22227d
actual(hex_decoded)   : {"pool_id":"0","routes":[],"token_out":""}

It is confused to reflect the returned payload after bump - it does not stores the result of simulation... So it'll be great to know what is actually happened in the core side!

@byeongsu-hong
Copy link
Author

Solved after bump to main branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant