Merge pull request #2549 from sebix/fix-2548

sql output: handle empty string fields paramter

Author: sebix

CHANGELOG.md

@@ -40,6 +40,8 @@
   - The requirement can only be installed on Python version < 3.12.
   - Add a check on the Python version and exit if incompatible.
   - Add a deprecation warning (PR#2544 by Sebastian Wagner)
+- `intelmq.bots.outputs.sql.output`:
+  - Treat an empty string `fields` parameter as unset parameter, fixing a crash in default configuration (PR#2548 by Sebastian Wagner, fixes #2548).
 
 ### Documentation
 - `docs/admin/installation/linux-packages`: Add `[signed-by=]` options, add wget command as alternative to curl (PR#2547 by Sebastian Wagner).

intelmq/bots/outputs/sql/output.py

@@ -43,9 +43,7 @@ def init(self):
     def process(self):
         event = self.receive_message().to_dict(jsondict_as_string=self.jsondict_as_string)
 
-        key_names = self.fields
-        if key_names is None:
-            key_names = event.keys()
+        key_names = self.fields or event.keys()
         valid_keys = [key for key in key_names if key in event]
         keys = '", "'.join(valid_keys)
         values = self.prepare_values(itemgetter_tuple(*valid_keys)(event))