diff --git a/pyproject.toml b/pyproject.toml index 18602d8ec..b221c8290 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -269,6 +269,9 @@ extend-select = [ "Q", # https://docs.astral.sh/ruff/rules/#flake8-quotes-q "RSE", # https://docs.astral.sh/ruff/rules/#flake8-raise-rse "RUF", # https://docs.astral.sh/ruff/rules/#ruff-specific-rules-ruf + "S102", # https://docs.astral.sh/ruff/rules/#flake8-bandit-s + "S110", # https://docs.astral.sh/ruff/rules/#flake8-bandit-s + "S112", # https://docs.astral.sh/ruff/rules/#flake8-bandit-s "SIM", # https://docs.astral.sh/ruff/rules/#flake8-simplify-sim "T100", # https://docs.astral.sh/ruff/rules/#flake8-debugger-t10 "T20", # https://docs.astral.sh/ruff/rules/#flake8-print-t20 diff --git a/src/documents/consumer.py b/src/documents/consumer.py index 970a259fd..fc13c959f 100644 --- a/src/documents/consumer.py +++ b/src/documents/consumer.py @@ -970,7 +970,7 @@ class ConsumerPlugin( try: copy_basic_file_stats(source, target) except Exception: # pragma: no cover - pass + self.log.debug("Unable to copy file stats from %s to %s", source, target) class ConsumerPreflightPlugin( diff --git a/src/documents/signals/handlers.py b/src/documents/signals/handlers.py index b76759781..c00be0e33 100644 --- a/src/documents/signals/handlers.py +++ b/src/documents/signals/handlers.py @@ -637,7 +637,7 @@ def update_filename_and_move_files( # so this is not the end of the world. # B: if moving the original file failed, nothing has changed # anyway. - pass + logger.exception("Error reverting document changes") # restore old values on the instance instance.filename = old_filename diff --git a/src/documents/views.py b/src/documents/views.py index c3e3c950b..cd1f1fd4f 100644 --- a/src/documents/views.py +++ b/src/documents/views.py @@ -1449,7 +1449,7 @@ class DocumentViewSet( try: lang = detect(doc.content) except Exception: - pass + logger.debug("Unable to detect language for document %s", doc.pk) meta["lang"] = lang return Response(meta)