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

soroban-cli: error messages saying to use config identity fund suggest a broken command #1014

Closed
leighmcculloch opened this issue Oct 7, 2023 · 0 comments · Fixed by #1020
Labels
bug Something isn't working

Comments

@leighmcculloch
Copy link
Member

leighmcculloch commented Oct 7, 2023

What version are you using?

20.0.0-rc3

What did you do?

  • 🕴🏻 Generated a new identity.
  • 🚀 Attempted to use it to deploy a contract.
  • 🤦🏻 See an error message saying the identity's account does't exist.
  • 💻 Ran the command the error message said to run.
soroban config identity generate mesoroban contract deploy --network local --source me --wasm contract.wasm
error: Account not found: GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF
Might need to fund account like:
soroban config identity fund GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF --network <name>
soroban config identity fund GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF --helper-url <url>soroban config identity fund GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF --network local
...

What did you expect to see?

  • Success!
soroban config identity fund GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF --network local

What did you see instead?

  • A message saying the identity can't be found.
soroban config identity fund GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF --network local
error: Failed to find config identity for GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF

Discussion

It seems that the error message thinks it should be possible to fund any G address. However the fund command appears to only accept an identity name, not a G address.

For example, the following command works:

soroban config identity fund me --network local

The immediate fix is probably to fix the error message.

But the better fix would be to "fix" the fund command so it lets you fund any G address.

@leighmcculloch leighmcculloch added the bug Something isn't working label Oct 7, 2023
willemneal added a commit to AhaLabs/stellar-cli that referenced this issue Oct 11, 2023
tsachiherman added a commit that referenced this issue Oct 16, 2023
fixes #1014

Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
willemneal added a commit to AhaLabs/stellar-cli that referenced this issue Oct 20, 2023
fixes stellar#1014

Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
willemneal added a commit to AhaLabs/stellar-cli that referenced this issue Oct 20, 2023
fixes stellar#1014

Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
tsachiherman added a commit that referenced this issue Oct 20, 2023
* Feat!(CLI): remove sandbox (#997)

* feat!: remove run_in_sandbox
* fix: reorganize tests that rely on network and ignore outside of go test
* fix: use feature instead of cfg so that cargo test --list can find tests
* fix: move common tests to single functions to skip setup time
* feat!: skip wasm in invoke
* fix: fully remove all sandbox related types and args

* fix: clippy --all (#1013)

* fix: clippy and git-hooks

* fix: split CLI tests and normal go tests so they can run concurrently

* fix(CLI): fund command now can accept a public strkey (#1020)

fixes #1014

Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>

---------

Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant