-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Labels
Description
Current State
- Many configuration values are hardcoded in code
- Default values scattered throughout codebase
- Missing configuration validation
Issues
- Cannot configure for different environments (dev, staging, prod)
- Hard to change settings without code changes
- No validation of configuration values
- Configuration errors only discovered at runtime
Required
- Externalize all configuration to config files or environment variables
- Add configuration validation on startup
- Document all configuration options with:
- Description
- Default value
- Valid range/format
- Required vs optional
- Provide configuration templates for different environments
- Fail fast on invalid configuration
Location
mobile-sensor-microservice/service.py(default values - lines 25-31, 68-74)tpm-plugin/tpm_plugin_server.py(default paths)- Test scripts (hardcoded IPs, ports)
Related
- [Enhancement] Externalize hardcoded configuration defaults #133 (Enhancement: Externalize hardcoded configuration defaults)
- From PRODUCTION_READINESS_ANALYSIS.md - Section 4.1