sickprodigy/Auto-Garden
1
0
Fork 0
Code Issues 4 Pull Requests Packages Projects Releases Wiki Activity

fix: Improve HTTP response handling and clarify default values in schedule configuration

Browse Source
This commit is contained in:
Aaron 2025-11-14 21:18:20 -05:00
parent 03766d6b09
commit 73b5a5aefe
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Scripts/web_server.py
View File

@ -50,7 +50,7 @@ class TempWebServer:
content_length = int(line.split(':')[1].strip())
break
# If POST request with body, read remaining data
# If POST request with body, read remaining data
if 'POST' in request and content_length > 0:
# Check how much body we already have
header_end = request.find('\r\n\r\n') + 4
@ -175,8 +175,13 @@ class TempWebServer:
elif 'GET /ping' in request:
# Quick health check endpoint (no processing)
conn.sendall(b'HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\nConnection: close\r\n\r\n')
conn.sendall(b'OK')
body = b'OK'
conn.sendall(b'HTTP/1.1 200 OK\r\n')
conn.sendall(b'Content-Type: text/plain\r\n')
conn.sendall(b'Content-Length: 2\r\n')
conn.sendall(b'Connection: close\r\n')
conn.sendall(b'\r\n')
conn.sendall(body)
conn.close()
return
@ -187,7 +192,7 @@ class TempWebServer:
response = self._get_status_page(sensors, ac_monitor, heater_monitor, schedule_monitor)
# ===== START: Send response with proper HTTP headers =====
print("DEBUG: Sending response ({} bytes)".format(len(response)))
print("DEBUG: Sending response ({} bytes)".format(len(response.encode('utf-8'))))
try:
# Check if response already has HTTP headers (like redirects)
if response.startswith('HTTP/1.1'):
@ -1379,8 +1384,8 @@ document.addEventListener('DOMContentLoaded', function() {{
schedules.append({
'time': '',
'name': '',
'ac_target': config.get('ac_target', 75.0), # ✅ Uses 78°F from config
'heater_target': config.get('heater_target', 72.0) # ✅ Uses 70°F from config
'ac_target': config.get('ac_target', 75.0), # default if not set
'heater_target': config.get('heater_target', 72.0) # default if not set
})
# ===== DEBUG: Verify we have 4 schedules =====