Re: [pmacct-discussion] [PATCH] Minor AMQP improvements

2015-04-01 Thread Nick Douma
Hi, On 01-04-15 05:18, Will Dowling wrote: I forgot to address the other (more important) bug I encountered: * Content type is only specified for messages published when the amqp_persistent_msg configuration option is specified. This information should always be applied to

Re: [pmacct-discussion] [PATCH] Minor AMQP improvements

2015-04-01 Thread Paolo Lucente
Hi Will, Thanks for both patches. They both make sense and will go in mainstream code. This second one i've already validated and is already applied to the code. The other one i just need some extra minimal time for QA (count it will be committed tomorrow).

Re: [pmacct-discussion] [PATCH] Minor AMQP improvements

2015-03-31 Thread Will Dowling
Whoopsy, I forgot to address the other (more important) bug I encountered: * Content type is only specified for messages published when the amqp_persistent_msg configuration option is specified. This information should always be applied to describe the payload of the message.