cdsapi icon indicating copy to clipboard operation
cdsapi copied to clipboard

Update api.py

Open si-bremer-enbw opened this issue 3 years ago • 2 comments

Updated default verify in Client class to True (instead of None)

The line "self.verify = True if verify else False" takes verify, if it is None, the result of this if clause will be False. Verify false is a bad default, disabling SSL verification as a default is a bad security practice.

si-bremer-enbw avatar Mar 02 '22 14:03 si-bremer-enbw

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

FussyDuck avatar Mar 02 '22 14:03 FussyDuck

This pr is related to https://github.com/ecmwf/cdsapi/pull/39

akeeman avatar Sep 14 '22 07:09 akeeman

This has been addressed by #64

EddyCMWF avatar Feb 27 '23 15:02 EddyCMWF