rick/LEDMatrix
1
0
Fork 0
mirror of https://github.com/ChuckBuilds/LEDMatrix.git synced 2026-04-13 05:53:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add API key validation and improve error handling in weather manager.

Browse Source
This commit is contained in:
ChuckBuilds
2025-04-19 17:27:50 -05:00
parent 508d01323a
commit 48909001e8
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/weather_manager.py
View File

@@ -128,6 +128,9 @@ class WeatherManager:
try: try:
# Fetch new data from OpenWeatherMap API # Fetch new data from OpenWeatherMap API
api_key = self.weather_config.get('api_key') api_key = self.weather_config.get('api_key')
if not api_key:
self.logger.error("No API key configured for OpenWeatherMap")
return None
# Construct full location string # Construct full location string
city = self.location.get('city') city = self.location.get('city')
@@ -161,6 +164,9 @@ class WeatherManager:
try: try:
self.logger.debug(f"Making request to {current_url} with params: {params}") self.logger.debug(f"Making request to {current_url} with params: {params}")
response = self.session.get(current_url, params=params, timeout=10) response = self.session.get(current_url, params=params, timeout=10)
if response.status_code == 401:
self.logger.error("Invalid API key for OpenWeatherMap")
return None
response.raise_for_status() response.raise_for_status()
current_data = response.json() current_data = response.json()
self.logger.debug(f"Current weather response: {current_data}") self.logger.debug(f"Current weather response: {current_data}")
@@ -178,6 +184,9 @@ class WeatherManager:
try: try:
self.logger.debug(f"Making request to {forecast_url} with params: {params}") self.logger.debug(f"Making request to {forecast_url} with params: {params}")
response = self.session.get(forecast_url, params=params, timeout=10) response = self.session.get(forecast_url, params=params, timeout=10)
if response.status_code == 401:
self.logger.error("Invalid API key for OpenWeatherMap")
return None
response.raise_for_status() response.raise_for_status()
forecast_data = response.json() forecast_data = response.json()
self.logger.debug(f"Forecast response: {forecast_data}") self.logger.debug(f"Forecast response: {forecast_data}")