Consider validating mapped properties

rabbitmq / rabbitmq-server

Open source RabbitMQ: core server and tier 1 (built-in) plugins

https://www.rabbitmq.com/

Other

12.15k stars 3.91k forks source link

Consider validating mapped properties #2593

Closed kjnilsson closed 3 weeks ago

kjnilsson commented 5 years ago

Some message properties are mapped directly to 0.9.1 properties but we don't validate that the types used are correct. For example the correlation id in 0.9.1 can only be a shortstr where in 1.0 it can be any AMQP 1.0 type.

https://github.com/rabbitmq/rabbitmq-amqp1.0/blob/3875953712ff14c0fef13288eadb121757b4f289/src/rabbit_amqp1_0_message.erl#L158

ansd commented 3 weeks ago

Fixed in RabbitMQ 4.0.