[Asterisk-code-review] main/stasis endpoints: Fix ContactStatusChange JSON for roun... (asterisk[master])

Matt Jordan asteriskteam at digium.com
Tue Nov 3 08:18:41 CST 2015


Matt Jordan has uploaded a new change for review.

  https://gerrit.asterisk.org/1555

Change subject: main/stasis_endpoints: Fix ContactStatusChange JSON for roundtrip_usec field
......................................................................

main/stasis_endpoints: Fix ContactStatusChange JSON for roundtrip_usec field

The JSON packing for the ContactStatusChange event forgot to include the
roundtrip_usec field. As a result, the field never showed up in any event,
even when the data was available. This patch corrects that error by properly
packing the JSON blob with the data.

Change-Id: I8df80da659a44010afbd48f645967518ff5daa17
---
M main/stasis_endpoints.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/55/1555/1

diff --git a/main/stasis_endpoints.c b/main/stasis_endpoints.c
index f44ce42..931bbb9 100644
--- a/main/stasis_endpoints.c
+++ b/main/stasis_endpoints.c
@@ -247,7 +247,7 @@
 		return NULL;
 	}
 
-	json_final = ast_json_pack("{s: s, s: o, s: o, s: { s: s, s: s, s: s } } ",
+	json_final = ast_json_pack("{s: s, s: o, s: o, s: { s: s, s: s, s: s, s: s } } ",
 		"type", "ContactStatusChange",
 		"timestamp", ast_json_timeval(*tv, NULL),
 		"endpoint", json_endpoint,

-- 
To view, visit https://gerrit.asterisk.org/1555
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8df80da659a44010afbd48f645967518ff5daa17
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list