Semir Patel comments

Results 24 comments of


                                            Semir Patel

Initial connection to Mi Flora sensor [...] failed (was working previously on a pi zero)

New user to miflora here and had same issue trying to get miflora working in a docker container on a rpi. Turns out the `--net host` is necessary (at least...

Make `RequestsClient` immutable.

This looks fixed in https://github.com/Yelp/bravado/pull/127

Rename SwaggerClient to BravadoClient?

Good idea. We can create an alias in a backward-compatible minor point release with `SwaggerClient` marked as deprecated. Then, remove `SwaggerClient` entirely in the next non-backward compatible major release.

Raise helpful message when `type` is not found for a definition

Both should be in the error message.

documentation does not indicate how to pass global headers

Wrap SwaggerClient to return a wrapped Resource that injects the headers via [`_request_options` ](http://bravado.readthedocs.io/en/latest/configuration.html#per-request-configuration)when the service call is made.

Add capacity to set a default timeout on the client

This is currently a work in progress as an internal library. See y/retries_and_timeouts.

Add capacity to set a default timeout on the client

Leaving open until a solution exists somewhere.

Unmarshaling responses more flexibly

I think this is a pretty important feature to have for more complicated use cases. Implementing it without breaking backwards compatibility might be hard to do. The return value of...

Bravado tries to parse xml response as JSON

That definitely needs fixing: https://github.com/Yelp/bravado-core/blob/master/bravado_core/response.py#L102

Bravado tries to parse xml response as JSON

swaggerpy (swagger 1.2) was refactored so much whilst adding Swagger 2.0 support (bravado/bravado-core) that it was effectively a rewrite. raw_response didn't make it in the final cut :cry: