fix: api expects lower case boolean value in querystring. Closes #32

ovh / python-ovh

Thin wrapper around OVH's APIs. Handles all the hard work including credential creation and requests signing.

https://pypi.org/project/ovh/

Other

298 stars 81 forks source link

fix: api expects lower case boolean value in querystring. Closes #32 #33

Closed yadutaf closed 8 years ago

yadutaf commented 8 years ago

OVH API expects Boolean values in the querystring to be sent as lower case 'true' or 'false' string. Such cases are typically represented as checkboxes in https://api.ovh.com/console. This PR adds the fix and the related regression test.

@ncrocfer Can you confirm this fixes your issue before I can merge?

ncrocfer commented 8 years ago

Yep, it's ok for me :

In [1]: import ovh

In [2]: ovh.Client().get('/hosting/web/ncrocfer.ovh/ovhConfig', historical=False)
Out[2]: [1779827]

Thanks :+1: