-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3776/
-----------------------------------------------------------
Review request for Asterisk Developers.
Bugs: ASTERISK-23985
https://issues.asterisk.org/jira/browse/ASTERISK-23985
Repository: Asterisk
Description
-------
This patch addresses two problems with the response to the PresenceState action:
(1) It calls astman_start_ack such that the ActionID, if present, is sent back
with the response
(2) It deprecates the second Message key in the response (which contains the
presence message). Responses often contain a key of "Message"; dual purposing
this key is cumbersome for many AMI clients that use dictionaries or hash
tables to store events/responses. This patch proposes that the response contain
the presence message in a PresenceMessage field. When the next major version of
AMI is made, the duplicate Message field should be removed.
Part (1) of this patch should be backported to 11/12.
Diffs
-----
/trunk/main/manager.c 418582
/trunk/UPGRADE.txt 418582
Diff: https://reviewboard.asterisk.org/r/3776/diff/
Testing
-------
Thanks,
Matt Jordan
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
