Review Board 1.7.16


cleanup dialog-info+xml Notify dialog

Review Request #1693 - Created Jan. 25, 2012 and submitted

Alec Davis
Asterisk 1.8 and above
Reviewers
asterisk-dev
Asterisk
Make consistant with the other dialog types, (and more human readable)
Simply added "\n" in 3 places

Sip debug previously looked like;

<?xml version="1.0"?><dialog-info xmlns="urn:ietf:params:xml:ns:dialog-info" version="715" state="full" entity="sip:8523@192.168.x.xx"><dialog id="8523">
<state>terminated</state>
</dialog>
</dialog-info>

Sip debug now looks like below;

User-Agent: Asterisk PBX 1.8.8.2
Subscription-State: active
Event: dialog
Content-Type: application/dialog-info+xml
Content-Length: 207

<?xml version="1.0"?>
<dialog-info xmlns="urn:ietf:params:xml:ns:dialog-info" version="715" state="full" entity="sip:8523@192.168.x.xx">
<dialog id="8523">
<state>terminated</state>
</dialog>
</dialog-info>
on a box with 116 registered hints.
Ship it!
Posted (Jan. 25, 2012, 2:58 p.m.)
Looks fine by me.

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.