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

console_wallet does not recognize password config setting, runtime error & exists #4404

Closed
hansieodendaal opened this issue Aug 5, 2022 · 3 comments
Assignees

Comments

@hansieodendaal
Copy link
Contributor

Describe the bug

The console wallet does not recognize the password config setting:

# Console wallet password. Should you wish to start your console wallet without typing in your password, the following
# options are available:
# 1. Start the console wallet with the --password=secret argument, or
# 2. Set the environment variable TARI_WALLET_PASSWORD=secret before starting the console wallet, or
# 3. Set the "password" key in this [wallet] section of the config
# (default = )
password = "1"

To Reproduce

Apply password in config.toml and run the console wallet.

Expected behavior

The wallet should not produce an error reading the password

Screenshots

There is an error in the configuration. Invalid value for `wallet`: invalid type: string "1", expected a sequence

Press any key to continue . . .

Desktop (please complete the following information):

Windows 10

Smartphone (please complete the following information):
n/a

Additional context

n/a

@agubarev agubarev self-assigned this Aug 8, 2022
@agubarev
Copy link
Contributor

agubarev commented Aug 8, 2022

tari-project/tari_utilities#47

This PR should fix this problem.

@SWvheerden
Copy link
Collaborator

SWvheerden commented Aug 15, 2022

This is still a problem

Initializing logging according to "C:\\...\\config\\log4rs_console_wallet.yml"
There is an error in the configuration. Invalid value for `wallet`: invalid type: string " ", expected a sequence

Press any key to continue . . .

stringhandler pushed a commit that referenced this issue Aug 30, 2022
Description
---
#4404

Motivation and Context
---
console_wallet does not recognize password config setting, runtime error & exists #4404

How Has This Been Tested?
---
manually (kudos to Stan)
@hansieodendaal
Copy link
Contributor Author

Tested, working fine!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

3 participants