davidvoit commented on PR #101: URL: https://github.com/apache/activemq-nms-amqp/pull/101#issuecomment-4236927380
Some background here: Java qpid only supports signed byte values or string (legacy mode in the source code). A reply to type is not correctly handled. Also the artemis webconsole doesn't like unsigned byte type. Here here the attribute is just silently hidden - this was quite hard to debug, as the easy to use debug tools, did show no annotation at all is set, instead of just the wrong data type. We will create seperate issues on qpid and artemis, but nontheless this is a bug within nms amqp. The spec says annotation type *must* by a byte, and bytes are definied in the amqp spec as signed bytes (just like a java byte type). The spec also says that a recieving client *must* support all integer types anyways, but this is a problem for qpid. As it's not that easy to replace all exisiting recieving client, we think this should be fixed on both implementation ends. Sender and Reciever. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information, visit: https://activemq.apache.org/contact
