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

Survey transaction error handling #12

Open
junha1 opened this issue Jul 20, 2022 · 7 comments
Open

Survey transaction error handling #12

junha1 opened this issue Jul 20, 2022 · 7 comments
Assignees

Comments

@junha1
Copy link
Member

junha1 commented Jul 20, 2022

No description provided.

@sms5030
Copy link

sms5030 commented Jul 22, 2022

@sms5030
Copy link

sms5030 commented Jul 22, 2022

@junha1
Copy link
Member Author

junha1 commented Jul 23, 2022

코스모스 모듈에 대해 전반적으로 이해하시고, 트랜잭션을

  1. Cosmos 레벨 (더 하이레벨)
  2. CosmWasm레벨 (더 스페시픽, Rust 코드레벨에서 어떻게 하는지 등)
    두 측면에서 모두 살펴봐야할 것 같습니다.

@sms5030
Copy link

sms5030 commented Jul 28, 2022

Tendermint Docs - CheckTx
https://docs.tendermint.com/master/spec/abci/abci.html#checktx

(이슈와 직접적인 연관이 있는 진 판단이 되지 않으나, 서치 히스토리를 남기기 위해 코멘트 남깁니다.)

@sms5030
Copy link

sms5030 commented Jul 28, 2022

https://docs.cosmos.network/v0.44/modules/
준하님이 피드백 해주신 cosmos module에 대한 문서를 찾아 히스토리를 남기고자 코멘트 남깁니다.

@sms5030
Copy link

sms5030 commented Jul 28, 2022

@sms5030
Copy link

sms5030 commented Jul 28, 2022

https://docs.cosmos.network/main/basics/tx-lifecycle.html#ValidateBasic
https://docs.cosmos.network/main/building-modules/msg-services.html#validation

  • cosmos에서 msg에 대해 validation check를 하는 기능이 msg.ValidateBasic()

  • 이는 CheckTx(Tendermint Core)를 이용해서 간단하게 (엄밀하지 않게) 체크함

  • 여기서 Invalid가 되도 수수료가 들지 않는다는 특징이 있다고 추측
    (Add guidelines for ValidateBasic cosmos/cosmos-sdk#10680 여기서 나온 내용이라 뇌피셜일 것 같습니다.)

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

2 participants