Skip to content

Commit

Permalink
Merge pull request #101 from slohse/pyyaml_safe_load
Browse files Browse the repository at this point in the history
Use PyYAML's safe_load instead of load
  • Loading branch information
matrulda committed Nov 25, 2021
2 parents 36070f4 + 5976a73 commit e44042e
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion checkQC/__init__.py
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@

__version__ = "3.6.5"
__version__ = "3.6.6"
4 changes: 2 additions & 2 deletions checkQC/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ def _get_config_file(config_path):
log.info("No config file specified, using default config from {}.".format(config_path))

with open(config_path) as stream:
return yaml.load(stream)
return yaml.safe_load(stream)
except FileNotFoundError as e:
log.error("Could not find config file: {}".format(e))
raise e
Expand All @@ -58,7 +58,7 @@ def get_logging_config_dict(config_path):
config_path = resource_filename(Requirement.parse('checkQC'), 'checkQC/default_config/logger.yaml')
log.info("No logging config file specified, using default config from {}.".format(config_path))
with open(config_path) as stream:
return yaml.load(stream)
return yaml.safe_load(stream)
except FileNotFoundError as e:
log.error("Could not find config file: {}".format(e))
raise e
Expand Down
2 changes: 1 addition & 1 deletion requirements/prod
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
click==6.7
PyYAML==3.12
PyYAML==6.0
interop>=1.1.10
xmltodict==0.11.0
sample_sheet
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
download_url='https://github.com/Molmed/checkQC/archive/{}.tar.gz'.format(__version__),
install_requires=[
"click",
"PyYAML>=3.12",
"PyYAML>=6.0",
"interop>=1.1.10",
"xmltodict",
"tornado",
Expand Down

0 comments on commit e44042e

Please sign in to comment.