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

Unused and missing tags on MsgCreateValidator #3793

Closed
4 tasks
fedekunze opened this issue Mar 4, 2019 · 0 comments
Closed
4 tasks

Unused and missing tags on MsgCreateValidator #3793

fedekunze opened this issue Mar 4, 2019 · 0 comments
Assignees
Milestone

Comments

@fedekunze
Copy link
Collaborator

fedekunze commented Mar 4, 2019

Summary of Bug

MsgCreateValidator currently has tags that imho are not necessary/useful: moniker and identity (this one also doesn't contain any value if the user doesn't provide any identity).

Besides, it's missing a delegator tag entry. Currently clients have to modify the bech32 prefix of a delegator to check for this type of transaction though the GET /txs endpoint.

Steps to Reproduce

{
  "height": "193281",
  "txhash": "DB8F2DAFD4791337E57921DB2D7D7838E14492323FECB391BB39D51239BA3FBF",
  "log": "[{\"msg_index\":\"0\",\"success\":true,\"log\":\"\"}]",
  "gas_wanted": "101587",
  "gas_used": "101577",
  "tags": [
    {
      "key": "action",
      "value": "create_validator"
    },
    {
      "key": "destination-validator",
      "value": "cosmosvaloper1qecshyc40kshszkwrtscgmsdd8tz3n4hrj9yf2"
    },
    {
      "key": "moniker",
      "value": "propelmind"
    },
    {
      "key": "identity"
    }
  ],
  "tx": {
    "type": "auth/StdTx",
    "value": {
      "msg": [
        {
          "type": "cosmos-sdk/MsgCreateValidator",
          "value": {
            "description": {
              "moniker": "propelmind",
              "identity": "",
              "website": "",
              "details": ""
            },
            "commission": {
              "rate": "0.100000000000000000",
              "max_rate": "0.200000000000000000",
              "max_change_rate": "0.010000000000000000"
            },
            "min_self_delegation": "1",
            "delegator_address": "cosmos1qecshyc40kshszkwrtscgmsdd8tz3n4hxx339e",
            "validator_address": "cosmosvaloper1qecshyc40kshszkwrtscgmsdd8tz3n4hrj9yf2",
            "pubkey": "cosmosvalconspub1zcjduepqjja8zg8ccvdd5rx50ad79faaaucq5vr98pr7eawnslqrscelwhaszhaq83",
            "value": {
              "denom": "muon",
              "amount": "1000000"
            }
          }
        }
      ],
      "fee": {
        "amount": null,
        "gas": "101587"
      },
      "signatures": [
        {
          "pub_key": {
            "type": "tendermint/PubKeySecp256k1",
            "value": "ArnKulECjCWw1IZNAj7UmGEeenfgTOPUmhGHnDwynYa3"
          },
          "signature": "t76SdT5xfg/ZzRopzrMPWINdGu3nOZnAql0ZuS5ANWdj+BI+dYJHMSt7hLID0eLm4VxNdhrR9VZ/L9yuFv2IGg=="
        }
      ],
      "memo": ""
    }
  }
}

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@fedekunze fedekunze self-assigned this Apr 1, 2019
@fedekunze fedekunze added this to the v0.35.0 milestone Apr 1, 2019
alessio pushed a commit that referenced this issue Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants