fix: handle multiple path parameters

exoscale / python-exoscale

Python bindings for the Exoscale APIs

https://exoscale.github.io/python-exoscale/

ISC License

15 stars 8 forks source link

fix: handle multiple path parameters #57

Closed thomas-chauvet closed 5 months ago

thomas-chauvet commented 5 months ago

When calling an API endpoint with multiple path parameters, it raised a KeyError because path parameters were handled one by one.

This PR fix this by handling query parameters the same as body parameters. It means looping through all parameters THEN formatting them for the endpoint URL.

For instance, it doesn't work for update_dbaas_pg_connection_pool and lot of others...

thomas-chauvet commented 5 months ago

Can you merge and release it? It will be nice, if not I can't use the library :)

brutasse commented 5 months ago

0.9.1 is up on PyPI, thanks again!