pound-python / infobob

ISC License
3 stars 2 forks source link

Make config more explicit and obvious #8

Open cdunklau opened 5 years ago

cdunklau commented 5 years ago

After #7 is fixed, configuration should be formalized somewhat to make it easier to see what's required, optional, etc. A few attrs classes should do the trick, perhaps with marshmallow or similar to do the config validation.

There is some leftover cruft in the example config file, e.g. database.dbm (for long-removed markov chain), misc.magic8_file, and the "divine" command in the channel ACLs.

There are also some required things missing from the config file, although these should probably be optional and just use defaults if not provided:

Tasks:

cdunklau commented 5 years ago

Config object appears not to be mutated externally except for the config_loc and dbpool attribute.

Extra attributes:

Config file is a JSON object, structured as follows: