Repository: anchore/vunnel (language: python)

Issue:
Disallow all bare try-except clauses
Today we have several [`noqa: E722`](https://www.flake8rules.com/rules/E722.html) statements as well as ignoring blank `noqa` statements ([here](https://github.com/anchore/vunnel/blob/16efe8a426ca0716a8eacf8ed09ca56410825263/pyproject.toml#L218) and [here](https://github.com/anchore/vunnel/blob/16efe8a426ca0716a8eacf8ed09ca56410825263/pyproject.toml#L221)). We should:
- not allow bare noqa statements, as this can hide a lot of issues
- not allow any E722 exceptions, as this is a popular antipattern in this codebase and has lead to production regressions

Additionally, all `except:` blocks should log the exception and some useful context, to make feed failures easier to diagnose from logs.



Discussion / hints:

Fix patch:
diff --git a/src/vunnel/utils/fdb.py b/src/vunnel/utils/fdb.py
index 7c05dc1f..f254c557 100644
--- a/src/vunnel/utils/fdb.py
+++ b/src/vunnel/utils/fdb.py
@@ -1,5 +1,3 @@
-# noqa
-
 """
 A file-based database, requires a writeable directory (path), which is usually
 a requirement for drivers. It can store any additional metadata, accessible  as


Test patch:
diff --git a/tests/quality/config.yaml b/tests/quality/config.yaml
index b59a2d5e..3bd3f950 100644
--- a/tests/quality/config.yaml
+++ b/tests/quality/config.yaml
@@ -119,6 +119,8 @@ tests:
         use_cache: true
       - name: wolfi
         use_cache: true
+    additional_trigger_globs:
+      - src/vunnel/utils/fdb.py
     images:
       - docker.io/anchore/test_images:java-56d52bc@sha256:10008791acbc5866de04108746a02a0c4029ce3a4400a9b3dad45d7f2245f9da
       - docker.io/anchore/test_images:npm-56d52bc@sha256:ba42ded8613fc643d407a050faf5ab48cfb405ad3ef2015bf6feeb5dff44738d