diff options
| author | Flavio Percoco <flaper87@gmail.com> | 2015-06-15 13:48:27 +0200 |
|---|---|---|
| committer | Flavio Percoco <flaper87@gmail.com> | 2015-06-15 23:16:57 +0200 |
| commit | c8845b4bef4248fad4a9da8dad0455495b2bc6fd (patch) | |
| tree | d39565c4aa6a529923bcf8d4c7517a465cadf0f1 | |
| parent | f06b19628fbb11a32797f1095eba7c07b6343433 (diff) | |
| download | oslo-messaging-c8845b4bef4248fad4a9da8dad0455495b2bc6fd.tar.gz | |
Use `inferred=True` by default
Rabbitmq's amqp1.0 plugin doesn't have support for vbin8, which is the
default encoding used by qpid-proton. In order to workaround this issue
we should use inferred=True.
Closes-bug: #1465409
Change-Id: Id265917244d7e152c5e13a10df367a3e59de8e50
| -rw-r--r-- | oslo_messaging/_drivers/protocols/amqp/driver.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/oslo_messaging/_drivers/protocols/amqp/driver.py b/oslo_messaging/_drivers/protocols/amqp/driver.py index 705a090..cf1c912 100644 --- a/oslo_messaging/_drivers/protocols/amqp/driver.py +++ b/oslo_messaging/_drivers/protocols/amqp/driver.py @@ -46,7 +46,9 @@ LOG = logging.getLogger(__name__) def marshal_response(reply=None, failure=None): # TODO(grs): do replies have a context? - msg = proton.Message() + # NOTE(flaper87): Set inferred to True since rabbitmq-amqp-1.0 doesn't + # have support for vbin8. + msg = proton.Message(inferred=True) if failure: failure = common.serialize_remote_exception(failure) data = {"failure": failure} @@ -67,7 +69,9 @@ def unmarshal_response(message, allowed): def marshal_request(request, context, envelope): - msg = proton.Message() + # NOTE(flaper87): Set inferred to True since rabbitmq-amqp-1.0 doesn't + # have support for vbin8. + msg = proton.Message(inferred=True) if envelope: request = common.serialize_msg(request) data = { |