mirror of
https://github.com/Dispatcharr/Dispatcharr.git
synced 2026-07-23 10:08:36 +00:00
21901498de
This update addresses issues related to geventpool database connection leaks by consistently invoking `close_old_connections()` in `finally` blocks within the backup scheduler, plugin repository refresh, and core notification synchronization processes. These changes ensure that idle connections are properly released, preventing resource exhaustion. HLS proxy server has been disabled (hasn't been used). Additionally, the proxy server cleanup logic has been enhanced to safely handle the absence of proxy instances. Tests have been updated to validate these improvements and ensure stability in high-load scenarios.
45 lines
1.6 KiB
Python
45 lines
1.6 KiB
Python
import logging
|
|
|
|
from django.apps import AppConfig
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
|
|
class BackupsConfig(AppConfig):
|
|
default_auto_field = "django.db.models.BigAutoField"
|
|
name = "apps.backups"
|
|
verbose_name = "Backups"
|
|
|
|
def ready(self):
|
|
"""Initialize backup scheduler on app startup."""
|
|
from dispatcharr.app_initialization import should_skip_initialization
|
|
|
|
# Skip if this is a management command, worker process, or dev server
|
|
if should_skip_initialization():
|
|
return
|
|
|
|
logger.debug("Syncing backup scheduler on app startup")
|
|
self._sync_backup_scheduler()
|
|
|
|
def _sync_backup_scheduler(self):
|
|
"""Sync backup scheduler task to database."""
|
|
from django.db import close_old_connections
|
|
|
|
from core.models import CoreSettings
|
|
from .scheduler import _sync_periodic_task, DEFAULTS
|
|
try:
|
|
# Ensure settings exist with defaults if this is a new install
|
|
CoreSettings.objects.get_or_create(
|
|
key="backup_settings",
|
|
defaults={"name": "Backup Settings", "value": DEFAULTS.copy()}
|
|
)
|
|
|
|
# Always sync the periodic task (handles new installs, updates, or missing tasks)
|
|
logger.debug("Syncing backup scheduler")
|
|
_sync_periodic_task()
|
|
except Exception as e:
|
|
# Log but don't fail startup if there's an issue
|
|
logger.warning(f"Failed to initialize backup scheduler: {e}")
|
|
finally:
|
|
# Cron sync reads system_settings for timezone; return geventpool checkouts.
|
|
close_old_connections()
|