Verify dynamic_queue query is respecting tags, fixed bug in static queue tag handling, added new link layer tag unit tests
Breaking Changes
-
GetNextQueuedMessageResponse(part ofGetNextQueuedMessage.srv) has a new field:string[] msg_tags.
New Features / Improvements
-
GetNextQueuedMessageResponsenew field:string[] msg_tags.-
string[] msg_tagsare the tags for the message returned in the response. - since dynamic_queues are not handled by the
message_queue_nodedirectly, we now use this field to validate responses. - when a response has invalid
msg_tags(logic defined by the current set ofrequire_tags,exclude_tagsand,minimum_priority) it is not used.
-
- added
deserialize_anymsg()tomac_utils.py
Bugfixes
-
TagConfigwas not settingself.exclude_queue_ids. it was settingset.exclude_list(typo!) and no other nodes were using that attr. -
message_queue_node.py:- queue response tag validation removes static network tags
forward,first_transmit,net_ackbefore validation - was not setting
minimum_prioritywhen creatingtag_configinstance. - was not passing
minimum_priorityto dynamic_queue_query call.
- queue response tag validation removes static network tags
Edited by Caileigh Fitzgerald