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

Add function to parse Figment-based configuration #22

Open
tlater-famedly opened this issue Mar 21, 2025 · 0 comments
Open

Add function to parse Figment-based configuration #22

tlater-famedly opened this issue Mar 21, 2025 · 0 comments
Labels
tech debt Work on this should be planned according to tech debt policy

Comments

@tlater-famedly
Copy link
Contributor

tlater-famedly commented Mar 21, 2025

Basically all our services now use figment for configuration, and all more or less write the same bits of code.

There is however one gnarly footgun in figment that we don't handle consistently, and the details of logging setup when a LogLevel is part of the configuration are also easy to miss. We should also assert that configuration priority (config before env before CLI) is done correctly in every repository, which we simply do not.

We should add a simple config parsing helper to this repository. It could even call process::exit() so that we do not need to care about error cases in config parsing elsewhere at all.

This would help by making configuration parsing take far less attention to detail, and increase consistency of how our services behave in front of end users.

@tlater-famedly tlater-famedly added the tech debt Work on this should be planned according to tech debt policy label Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech debt Work on this should be planned according to tech debt policy
Projects
None yet
Development

No branches or pull requests

1 participant