generic csv: handle empty string required_columns

fixes #2679

Author: sebix

CHANGELOG.md

@@ -25,6 +25,7 @@ Please refer to the [NEWS](NEWS.md) for a list of changes which have an affect o
 #### Collectors
 
 #### Parsers
+- `intelmq.bots.parsers.generic_csv.parser_csv`: Handle empty string parameter `columns_required` as unset (PR#2680 by Sebastian Wagner, fixes #2679).
 
 #### Experts
 

intelmq/bots/parsers/generic/parser_csv.py

@@ -68,7 +68,7 @@ def init(self):
                                   expected=("blacklist", "whitelist"),
                                   docs=DOCS)
 
-        if self.columns_required is None:
+        if self.columns_required in [None, '']:
             self.columns_required = [True for _ in self.columns]
         if len(self.columns) != len(self.columns_required):
             raise ValueError("Length of parameters 'columns' (%d) and 'columns_required' (%d) "

intelmq/tests/bots/parsers/generic/test_parser_csv.py

@@ -105,6 +105,12 @@ def test_compose_fields(self):
                                  'delimiter': ','})
         self.assertMessageEqual(0, COMPOSE_EVENT)
 
+    def test_empty_required_colums(self):
+        """ #2679 """
+        self.run_bot(parameters={'columns_required': ''})
+        self.assertMessageEqual(0, EXAMPLE_EVENT)
+        self.run_bot(parameters={'columns_required': None})
+
 
 if __name__ == '__main__':  # pragma: no cover
     unittest.main()