Skip to content

Commit 31c8bc4

Browse files
committed
Refactor logging in scheduler engine to use debug level for time-related messages, enhancing log clarity and reducing noise. Update developer credit in topbar for consistency.
1 parent b2d2265 commit 31c8bc4

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

frontend/templates/components/topbar.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
<div class="version-divider"></div>
2424
<div class="version-item">
2525
<i class="fas fa-code version-icon"></i>
26-
<span class="developer-credit">Developer: <a href="https://github.com/Admin9705/" target="_blank">Admin9705</a></span>
26+
<span class="developer-credit">Dev: <a href="https://github.com/Admin9705/" target="_blank">Admin9705</a></span>
2727
</div>
2828
</div>
2929
</div>

src/primary/scheduler_engine.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -385,7 +385,7 @@ def should_execute_schedule(schedule_entry):
385385
exact_time = datetime.datetime.now(user_tz)
386386
timezone_name = str(user_tz)
387387
time_with_tz = f"{exact_time.strftime('%Y-%m-%d %H:%M:%S.%f')} {timezone_name}"
388-
scheduler_logger.info(f"EXACT CURRENT TIME: {time_with_tz}")
388+
scheduler_logger.debug(f"EXACT CURRENT TIME: {time_with_tz}")
389389

390390
if not schedule_entry.get("enabled", True):
391391
scheduler_logger.debug(f"Schedule {schedule_id} is disabled, skipping")
@@ -399,7 +399,7 @@ def should_execute_schedule(schedule_entry):
399399
current_day = datetime.datetime.now(user_tz).strftime("%A").lower() # e.g., 'monday'
400400

401401
# Debug what's being compared
402-
scheduler_logger.info(f"CRITICAL DEBUG - Today: '{current_day}', Schedule days: {days}")
402+
scheduler_logger.debug(f"CRITICAL DEBUG - Today: '{current_day}', Schedule days: {days}")
403403

404404
# If days array is empty, treat as "run every day"
405405
if not days:
@@ -410,10 +410,10 @@ def should_execute_schedule(schedule_entry):
410410

411411
# If today is not in the schedule days, skip this schedule
412412
if current_day not in lowercase_days:
413-
scheduler_logger.info(f"FAILURE: Schedule {schedule_id} not configured to run on {current_day}, skipping")
413+
scheduler_logger.debug(f"FAILURE: Schedule {schedule_id} not configured to run on {current_day}, skipping")
414414
return False
415415
else:
416-
scheduler_logger.info(f"SUCCESS: Schedule {schedule_id} IS configured to run on {current_day}")
416+
scheduler_logger.debug(f"SUCCESS: Schedule {schedule_id} IS configured to run on {current_day}")
417417

418418

419419
# Get current time with second-level precision for accurate timing (in user's timezone)
@@ -442,19 +442,19 @@ def should_execute_schedule(schedule_entry):
442442
if timezone_name:
443443
time_debug_str += f" {timezone_name}"
444444

445-
scheduler_logger.info(f"Schedule {schedule_id} time: {schedule_hour:02d}:{schedule_minute:02d}, "
445+
scheduler_logger.debug(f"Schedule {schedule_id} time: {schedule_hour:02d}:{schedule_minute:02d}, "
446446
f"current time: {time_debug_str}")
447447

448448
# ===== STRICT TIME COMPARISON - PREVENT EARLY EXECUTION =====
449449

450450
# If current hour is BEFORE scheduled hour, NEVER execute
451451
if current_time.hour < schedule_hour:
452-
scheduler_logger.info(f"BLOCKED EXECUTION: Current hour {current_time.hour} is BEFORE scheduled hour {schedule_hour}")
452+
scheduler_logger.debug(f"BLOCKED EXECUTION: Current hour {current_time.hour} is BEFORE scheduled hour {schedule_hour}")
453453
return False
454454

455455
# If same hour but current minute is BEFORE scheduled minute, NEVER execute
456456
if current_time.hour == schedule_hour and current_time.minute < schedule_minute:
457-
scheduler_logger.info(f"BLOCKED EXECUTION: Current minute {current_time.minute} is BEFORE scheduled minute {schedule_minute}")
457+
scheduler_logger.debug(f"BLOCKED EXECUTION: Current minute {current_time.minute} is BEFORE scheduled minute {schedule_minute}")
458458
return False
459459

460460
# ===== 4-MINUTE EXECUTION WINDOW =====
@@ -475,7 +475,7 @@ def should_execute_schedule(schedule_entry):
475475
return True
476476

477477
# We've missed the 4-minute window
478-
scheduler_logger.info(f"MISSED WINDOW: Current time {current_time.hour:02d}:{current_time.minute:02d} "
478+
scheduler_logger.debug(f"MISSED WINDOW: Current time {current_time.hour:02d}:{current_time.minute:02d} "
479479
f"is past the 4-minute window for {schedule_hour:02d}:{schedule_minute:02d}")
480480
return False
481481

0 commit comments

Comments
 (0)