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

Matt Jordan asteriskteam at digium.com
Tue Nov 3 15:38:11 CST 2015


Matt Jordan has submitted this change and it was merged.

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(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/main/stasis_endpoints.c b/main/stasis_endpoints.c
index 157f98c..fbca2ad 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/1554
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8df80da659a44010afbd48f645967518ff5daa17
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list