🐛 fix(manager.py): catch and log exceptions when tearing down services to prevent silent failures
🐛 fix(utils.py): catch and log exceptions when tearing down services and superuser to prevent silent failures
This commit is contained in:
parent
e72c04a94a
commit
f9d9adf60b
2 changed files with 7 additions and 7 deletions
|
|
@ -90,7 +90,10 @@ class ServiceManager:
|
||||||
if service is None:
|
if service is None:
|
||||||
continue
|
continue
|
||||||
logger.debug(f"Teardown service {service.name}")
|
logger.debug(f"Teardown service {service.name}")
|
||||||
|
try:
|
||||||
service.teardown()
|
service.teardown()
|
||||||
|
except Exception as exc:
|
||||||
|
logger.exception(exc)
|
||||||
self.services = {}
|
self.services = {}
|
||||||
self.factories = {}
|
self.factories = {}
|
||||||
self.dependencies = {}
|
self.dependencies = {}
|
||||||
|
|
@ -150,9 +153,3 @@ def initialize_session_service():
|
||||||
dependencies=[ServiceType.CACHE_SERVICE],
|
dependencies=[ServiceType.CACHE_SERVICE],
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
def teardown_services():
|
|
||||||
"""
|
|
||||||
Teardown all the services.
|
|
||||||
"""
|
|
||||||
service_manager.teardown()
|
|
||||||
|
|
|
||||||
|
|
@ -153,6 +153,9 @@ def teardown_services():
|
||||||
"""
|
"""
|
||||||
try:
|
try:
|
||||||
teardown_superuser(get_settings_service(), next(get_session()))
|
teardown_superuser(get_settings_service(), next(get_session()))
|
||||||
|
except Exception as exc:
|
||||||
|
logger.exception(exc)
|
||||||
|
try:
|
||||||
service_manager.teardown()
|
service_manager.teardown()
|
||||||
except Exception as exc:
|
except Exception as exc:
|
||||||
logger.exception(exc)
|
logger.exception(exc)
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue