Fix logging configuration issue in config.py #24
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
updated the setup.md referral link
- Replace all print() statements with structured logging using Python logging module - Add centralized logging configuration (config/logging_config.py) with factory pattern - Implement dual output handlers: console (simple format) and file (detailed format) - Add log rotation using RotatingFileHandler (10MB files, 5 backups by default) - Support multiple log levels: DEBUG, INFO, WARNING, ERROR, CRITICAL - Add comprehensive CLI argument parsing for runtime configuration - Support environment variable configuration for containerized deployments - Include detailed file logging with module:function:line information - Add utility functions for common logging patterns (function entry/exit, exceptions) - Maintain backward compatibility with existing application functionality Features: - Configurable via CLI args (--log-level, --log-dir, --view-logs, --no-console-logging) - Environment variable support (LOG_LEVEL, LOG_DIR, LOG_MAX_BYTES, etc.) - Professional log formatting with timestamps and context information - Separate formatters for console (readable) and file (detailed debugging) - Comprehensive error handling and application flow logging Closes #original-issue: Professional logging implementation with configurable verbosity levels
Replace hardcoded credentials with config file (fixes 01bps#3)
Feat: Add Command-Line Interface
feat: implement professional logging system with dual output and rotation
Add web dashboard feature
Add Multi-Network-Support
|
This update fixes the logging configuration issue in the Changes made:
This improvement enhances debugging and monitoring of the WiFi-Auto-Auth application. |
|
@Girish200127 Kindly resolve the merge conflicts |
|
@Girish200127 have you gone through the issue properly. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This update fixes the logging configuration issue in the
config.pyfile of the WiFi-Auto-Auth project.Changes made:
logging_config.py
This improvement enhances debugging and monitoring of the WiFi-Auto-Auth application.