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

fix: invalid forced sync peer now returns configerror #3350

Conversation

StriderDM
Copy link
Contributor

@StriderDM StriderDM commented Sep 15, 2021

Description

Fixed bug which returned an UnknownError instead of a ConfigError for invalid force sync peers.
Remove double print of error.

Output:

Running `target\debug\tari_base_node.exe`
Initializing logging according to "C:\\Users\\DMStrider\\.tari\\config\\log4rs_base_node.yml"
Please check your force sync peers configuration
ConfigError("Invalid force sync peer: Invalid public key string")
14:43 ERROR Exiting with code (101): ConfigError("Invalid force sync peer: Invalid public key string")

Motivation and Context

Previously an UnknownError was reported for invalid force sync peers

How Has This Been Tested?

Configuration with invalid force_sync_peers.

Fixed bug which returned an UnknownError instead of a ConfigError for invalid force sync peers.
Remove double print of error.
@aviator-app aviator-app bot merged commit 8163ef8 into tari-project:development Sep 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants