Skip to content
This repository has been archived by the owner on Jun 3, 2020. It is now read-only.

tmkms.toml: Make [validator.addr] a URL #106

Merged
merged 1 commit into from
Nov 20, 2018
Merged

tmkms.toml: Make [validator.addr] a URL #106

merged 1 commit into from
Nov 20, 2018

Conversation

tony-iqlusion
Copy link
Contributor

@tony-iqlusion tony-iqlusion commented Nov 20, 2018

This change simplifies the configuration file by collapsing the large "connection" section of the configuration down to individual URLs which use the same syntax as in gaia, i.e. the [validator.addr] should be the same as gaiad's priv_validator_laddr

This change simplifies the configuration file by collapsing the large
"connection" section of the configuration down to individual URLs which
use the same syntax as in gaia.
use error::{KmsError, KmsErrorKind::*};

#[derive(Clone, Debug)]
pub enum ValidatorAddr {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This type is arguably useful as an address for more things than just validators, and could potentially go in the tendermint crate.

@tarcieri tarcieri requested review from liamsi and zmanian November 20, 2018 20:21
Copy link
Contributor

@liamsi liamsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tony-iqlusion!

@tarcieri tarcieri merged commit e186ed1 into master Nov 20, 2018
@tarcieri tarcieri deleted the simplify-config branch November 20, 2018 21:36
@tony-iqlusion tony-iqlusion mentioned this pull request Nov 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants