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

Signing transactions with Keplr error #128

Closed
fadeev opened this issue Nov 8, 2021 · 7 comments
Closed

Signing transactions with Keplr error #128

fadeev opened this issue Nov 8, 2021 · 7 comments
Assignees

Comments

@fadeev
Copy link
Contributor

fadeev commented Nov 8, 2021

I'm facing a weird issue, where I'm able to send transactions via the VUE app, but as soon as I try using the Keplr is fails with a error 501.

The front end shows:

Could not broadcast Tx: Transaction with ID 69FF1824CD271879CB9D3E2901CFB40DB1FF9AA115D1CFB15DEAF62C8073DDCE was submitted but was not yet found on the chain. You might want to check later

unknown

There is a full node connected to the validators, getting data from them... The urls on the VUE app are:

VUE_APP_API_COSMOS=https://rest-ufo.alien.blue
VUE_APP_WS_TENDERMINT=wss://wsrpc-ufo.alien.blue/websocket
VUE_APP_API_TENDERMINT=https://rpc-ufo.alien.blue
VUE_APP_ADDRESS_PREFIX=alien
VUE_APP_CHAIN_ID=moon

I think Kepler is expecting different endpoints maybe? does it really matter? It's def an issue with Kepler but I'm not sure if you or anyone else has encountered this issue.

cc @daniel-farina

@daniel-farina
Copy link

daniel-farina commented Nov 8, 2021

I'm adding more details here to reproduce the issue. I'm adding some seed phrases for the team to test it. This is fine as this chain will get deleted.

This is an out of the box vue app from starport.

The .env for the VUE app is:

VUE_APP_API_COSMOS=https://rest-ufo.alien.blue
VUE_APP_WS_TENDERMINT=wss://wsrpc-ufo.alien.blue/websocket
VUE_APP_API_TENDERMINT=https://rpc-ufo.alien.blue
VUE_APP_ADDRESS_PREFIX=alien
VUE_APP_CHAIN_ID=alien
VUE_APP_CHAIN_NAME="Alien"

What works

Sending from bash works without any issues:

Chain-id: alien

aliend tx bank send alien1ajay55sd4474vrs97fesqky9wt0rf9epyfw4ce alien1dut3t6c4m3m4zck7nr8mdm6an56xls26rw5yft 1000alien --chain-id alien --keyring-backend test

Also sending from the Vuejs's native wallet works perfectly.

The error - Steps to reproduce:

Background

I deleted the local store, and create a brand new Keplr wallet, new browser session, etc.

Visit: https://dapp.alien.blue/

Load this wallet to Keplr:

awesome problem genre agree paddle field banana witness wink business ask term hair doctor tree pond churn burden series involve swift author stem today

Now you are a millionare, jk.

Then connect the wallet like always
image

Then send 1000 to this address

alien1dut3t6c4m3m4zck7nr8mdm6an56xls26rw5yft

image

This what the payload looks like:

{
  "txBody": {
    "messages": [
      {
        "fromAddress": "alien1ajay55sd4474vrs97fesqky9wt0rf9epyfw4ce",
        "toAddress": "alien1dut3t6c4m3m4zck7nr8mdm6an56xls26rw5yft",
        "amount": [
          {
            "denom": "alien",
            "amount": "10000"
          }
        ]
      }
    ],
    "memo": ""
  },
  "authInfo": {
    "signerInfos": [
      {
        "publicKey": {
          "type_url": "/cosmos.crypto.secp256k1.PubKey",
          "value": "CiEDMUDoTCc+uaWzRAOi37p3v3/A4zNshFlbBlIES5qwvNs="
        },
        "modeInfo": {
          "single": {
            "mode": "SIGN_MODE_DIRECT"
          }
        },
        "sequence": "7"
      }
    ],
    "fee": {
      "amount": [
        {
          "denom": "alien",
          "amount": "5000"
        }
      ],
      "gasLimit": "200000"
    }
  },
  "chainId": "alien",
  "accountNumber": "0"
}

It will sit there processing for a while.....
image

Then this error shows on the console:

image

Also, if you send a transaction from Kepler:

image

It shows a error as well:

image

@marinhoarthur
Copy link
Contributor

We'll be experimenting with no-legacy-stdTx in here

As soon as we have updates on this, will be posted here.

cc @daniel-farina

@daniel-farina
Copy link

Great,
I was just testing that last night based on 167.

You should be able to use the .env above and run the vue locally against the testing chain we made.

Thank you

@marinhoarthur
Copy link
Contributor

marinhoarthur commented Nov 18, 2021

Adding the no-legacy-stdTx flag works. We'll prepare a release of the frontend. Soon enough it's deployed and I'll update this issue.

@daniel-farina
Copy link

Great news! We will test it on our side.

@marinhoarthur
Copy link
Contributor

marinhoarthur commented Nov 24, 2021

Released under v0.1.55

Updating your @starport/vue package is necessary to get the fix.

cc @daniel-farina

@daniel-farina
Copy link

awesome job @marinhoarthur
Thank you for working with us and keeping us posted along the way.

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

3 participants