Add request timeout setting to clients

Majsvaffla commented 5 years ago

Added a keyword argument request_timeout to each of the client classes. It's value defaults to None.

The SOAP client passes the argument on to the zeep transport instance it uses unless the value is None; then it doesn't pass it on because zeep has a default timeout built-in that would otherwise be overridden.
The private method _post is introduced on the JSON client. It should be used instead of using the post method on the requests session directly.
The JSON client keeps the value of the timeout argument around for use by the _post method which passes it to the requests session instance.

Fixes #20

hbldh commented 5 years ago

Sorry, this took longer time to get to than I expected. It is merged and released in version 0.9.0. Thank you!

Majsvaffla commented 5 years ago

Sorry, this took longer time to get to than I expected. It is merged and released in version 0.9.0. Thank you!

Great! No worries :)

hbldh / pybankid