Skip to content

Improve logging for better debugging#234

Open
daedalist wants to merge 3 commits intobohdan-s:mainfrom
daedalist:improve-error-logging
Open

Improve logging for better debugging#234
daedalist wants to merge 3 commits intobohdan-s:mainfrom
daedalist:improve-error-logging

Conversation

@daedalist
Copy link
Copy Markdown
Contributor

Summary

Enhances logging across multiple modules to improve debugging and troubleshooting capabilities.

Changes

Logging Improvements

  • sungather.py: Added detailed exception logging with stack traces for export failures
  • mqtt.py: Improved connection status logging and error messages
  • influxdb.py: Enhanced error logging for write operations
  • webserver.py: Added comprehensive logging for HTTP server lifecycle and request handling

Configuration

  • .gitignore: Added .private/ directory for local debugging notes and development files

Benefits

  • Better visibility into export failures with full stack traces
  • Easier troubleshooting of MQTT connection issues
  • Improved debugging of InfluxDB write errors
  • Clearer HTTP server status messages
  • Support for local development files without cluttering git history

Testing

  • Changes tested locally
  • No functional changes to core logic
  • Logging output verified during development

Files Changed

  • SunGather/sungather.py - Enhanced exception logging
  • SunGather/exports/mqtt.py - Improved MQTT connection logging
  • SunGather/exports/influxdb.py - Better InfluxDB error messages
  • SunGather/exports/webserver.py - Comprehensive HTTP server logging
  • .gitignore - Added .private/ directory

🤖 Coded with Claude Code assistance

@daedalist daedalist marked this pull request as ready for review October 19, 2025 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant