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

[FEAT] Use configuration validation patterns for environment based config #125

Open
belthesar opened this issue Feb 23, 2023 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@belthesar
Copy link
Contributor

Is your feature request related to a problem? Please describe.
TAU currently assumes that configuration received via environment variables will match the intended behavior. This can lead to silent failures, especially when deploying outside of application defaults, which assume a local deploy with Ngrok.

Describe the solution you'd like
In support of good documentation in both code and readmes, TAU should validate config, and raise exceptions as needed to inform the user deploying TAU of any issues where config is not in spec.

Describe alternatives you've considered
Documentation alone could be used to handle this in lieu of validating config, but will lead to silent errors / magic behavior.

Additional context
Consider using django-environ and/or environs to handle environment variable parsing instead of the existing use of os.environ

@belthesar belthesar added the enhancement New feature or request label Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant