jasima/majuna
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

lint: excuse catching broad exceptions

Browse source
This commit is contained in:
Iain Learmonth 2022-05-17 09:03:43 +01:00
parent 9797d8d119
commit fce594bbc4
2 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

8
app/cli/automate.py
View file

@ -85,14 +85,18 @@ def run_job(job_cls: Type[BaseAutomation], *,
logging.warning("Not time to run this job yet")
return
if not automation.enabled and not force:
logging.warning(f"job {job_cls.short_name} is disabled and --force not specified")
logging.warning("job %s is disabled and --force not specified", job_cls.short_name)
return
automation.state = AutomationState.RUNNING
db.session.commit()
job: BaseAutomation = job_cls()
try:
success, logs = job.automate()
except Exception as e:
# We want to catch any and all exceptions that would cause problems here, because
# the error handling process isn't really handling the error, but rather causing it
# to be logged for investigation. Catching more specific exceptions would just mean that
# others go unrecorded and are difficult to debug.
except Exception as e: # pylint: disable=broad-except
tb = TracebackException.from_exception(e)
success = False
logs = "".join(tb.format())

4
app/cli/db.py
View file

@ -68,7 +68,9 @@ def impot(model: db.Model) -> None:
db.session.add(x)
db.session.commit()
logging.info("Import completed successfully")
except Exception as e:
# Many things can go wrong in the above, like IO, format or database errors.
# We catch all the errors and ensure the database transaction is rolled back, and log it.
except Exception as e: # pylint: disable=broad-except
logging.exception(e)
db.session.rollback()