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

Fix missing cdc on ante handler options #25

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arifintahu
Copy link
Contributor

Background:
This issue has been fixed before but on the recent uptick release the cdc on ante handler options is missing again. It can cause error panic when executing authz Msg Exec #18.

Reproduce:

uptickd keys add mykey
uptickd keys add test
uptickd tx authz grant $(uptickd keys show test -a) send --spend-limit 1000000000000auptick --from mykey -y
uptickd q authz grants-by-grantee $(uptickd keys show test -a)

uptickd keys add recipient
uptickd tx bank send mykey $(uptickd keys show recipient -a) 1000auptick --from test --generate-only | jq > tx-send.json
uptickd tx authz exec tx-send.json --from test -y

Before:
uptick-before

After fixing:
uptick-after

@uptSmart
Copy link
Collaborator

uptSmart commented Feb 3, 2024

Thank you for your submission. We will confirm and test it first. If there are no problems, we will merge it into the 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

Successfully merging this pull request may close these issues.

None yet

2 participants