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

[Bug]: sending transaction - error #17923

Closed
1 task done
golnar-m opened this issue Sep 28, 2023 · 3 comments
Closed
1 task done

[Bug]: sending transaction - error #17923

golnar-m opened this issue Sep 28, 2023 · 3 comments
Labels
S:needs more info This bug can't be addressed until more information is provided by the reporter. T:question

Comments

@golnar-m
Copy link

golnar-m commented Sep 28, 2023

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

A Security bug happened!

Cosmos SDK Version

v0.45.5

How to reproduce?

I am trying to send a transaction:

soarchaind tx poa register-client 3059301306072a8648ce3d020106082a8648ce3d030107034200046c28e2efdf94600435dbba5ae7f195cb619e3dd128b7e0e2877f9a1da489027819001c3e0141cb579dc3d9e913a45644401bd2458313dc37d15dd58adcaff154 --from soar1ghfnkjlc5gxpldat7hm50tgggwc6l5h7ydwy2a --gas auto --gas-adjustment 1.5 --gas-prices 0.0025udmotus

But got this error:

Error: key with address45D33B4BF8A20C1FB7ABF5F747AD0843B1AFD2FEnot found: key not found

The address that I'm trying to send has balance :

**balances:

  • amount: "1500"**

But when I define my account as a keys into makefile by using below code, error doesn't happen. As you know below command asks for mnemonic.

keys:
	soarchaind keys add client --recover --keyring-backend test --algo secp256k1

I do not want to define my account in makefile as keys because this means I should put private key of the account into the node. So how can prevent this issue?

Actually we need a way send all txs without defining accounts private key/ mnemonics into the chain.

@golnar-m golnar-m added the T:Bug label Sep 28, 2023
@golnar-m golnar-m changed the title [Bug]: Sending transaction error [Bug]: Security bug in sending transaction - error Sep 29, 2023
@tac0turtle
Copy link
Member

why do you need to set the address in the makefile? can you share what you are doing in order for us to help you

@julienrbrt julienrbrt changed the title [Bug]: Security bug in sending transaction - error [Bug]: sending transaction - error Sep 30, 2023
@julienrbrt
Copy link
Member

Are you saying you fail to send a transaction of an account that isn't in the keyring?

For sending and thus signing a transaction you need to import the account you want to sign with. This will be added in the keyring.

Putting it in the makefile isn't needed.

@julienrbrt julienrbrt added S:needs more info This bug can't be addressed until more information is provided by the reporter. T:question and removed T:Bug labels Sep 30, 2023
@tac0turtle
Copy link
Member

closing due to staleness, please comment if you still need help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S:needs more info This bug can't be addressed until more information is provided by the reporter. T:question
Projects
None yet
Development

No branches or pull requests

3 participants