Closed barbich closed 5 months ago
Hi, thanks for catching that. The safe_load
is the right solution here, would you like to create a PR to change it?
option 3: use ruaml.yaml as has been done in the rest of the codebase
would it not be better for consistency reasons ?
I missed it, but yeah, it would be better :)
Hi
Environment
running: Latest (as of March2024) docker compose of intelmq.
Description
Call to intelmq_gen_feeds_conf fails due to deprecation of "Use of PyYAML's yaml.load function without specifying the Loader=... parameter, has been deprecated. In PyYAML version 5.1+, you will get a warning, but the function will still work." See https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation
For example:
Fix
replace yaml.load(fp.read()) by: