Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Errors when loading the configuration file should be easier to read #2491

Open
sandhose opened this issue Mar 18, 2024 · 1 comment
Open
Labels
A-Configuration Related on what is configurable and how it can be configured

Comments

@sandhose
Copy link
Member

sandhose commented Mar 18, 2024

Because we use figment to handle configuration, we sometimes have really weird error messages. See #2477 for example.

A few things we should do:

  • avoid #[serde(flatten)], as it breaks error reporting
  • tag important places of the config so that we can
  • do configuration post-processing instead
  • actually test some misconfiguration scenarios
@matrixbot
Copy link
Member

For your information, this issue has been copied over to the Element fork of matrix-authentication-service: element-hq/matrix-authentication-service#2491

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Configuration Related on what is configurable and how it can be configured
Projects
None yet
Development

No branches or pull requests

2 participants