consul-esm icon indicating copy to clipboard operation
consul-esm copied to clipboard

Config Struct Contains Unconfigurable Fields

Open lornasong opened this issue 4 years ago • 0 comments

Currently configuration is split between the Config struct and the HumanConfig struct where HumanConfig is a subset of Config fields that are actually configurable by the practitioner. The HumanConfig values are then merged with the Config struct that is then consumed.

This set-up can be confusing as not all fields in Config are actually configurable. New configurable configs also have to be added into both structs.

One possible solution would be to refactor out non-configurable options out of Config and into their own variables and remove the need for HumanConfig.

lornasong avatar Jun 08 '20 16:06 lornasong