Hi Paolo,

Small patch to help address some rough edges on the AMQP plugin, specifically:

    * Added some helpful error messages when handling unexpected replies
      from the AMQP server. This is particularly helpful when the plugin
      fails to re-declare an existing exchange with different parameters.

      It is possible to get more detailed information (there's an example in
      tools/common.c in the librabbitmq source), but I felt that this is a
      good compromise between my time/capability and bloating the code.

    * Generate an error on compile if —enable-rabbitmq is specified
      without —enable-jansson. It's clear in the documentation that both
      are required for AMQP support, but if built without jansson it will
      silently not publish messages to AMQP.

Apologies for the quality (lack thereof) of the patch, I'm not a C guy by
trade - hope it helps others out there. Maybe next time I’ll RTFM too ;)

Cheers :)


Will Dowling

Attachment: amqp-improvements.patch
Description: Binary data

_______________________________________________
pmacct-discussion mailing list
http://www.pmacct.net/#mailinglists

Reply via email to