[Home]

Summary:ASTERISK-04661: [patch] Adding cleartext hangupcause to BYE and CANCEL
Reporter:Olle Johansson (oej)Labels:
Date Opened:2005-07-23 12:23:39Date Closed:2008-01-15 15:42:22.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) siphangupcause.txt
Description:To make debugging easier, this patch adds cleartext hangupcauses to the SIP bye and CANCEL packets. At some point, we might want to add the code so that we can parse it on the other end.
Comments:By: twisted (twisted) 2005-07-23 14:47:35

I like this.  This would also work well on the remote side (assuming that we can read X-asterisk-hangupcause, which other asterisk/ser can), to determine why the call was hung up, and report it in the cdr appropriately, or in a call resolution db.  Great job Olle!
By: Olle Johansson (oej) 2005-07-23 16:08:13

Thanks for a positive reply. I can add another header, since you are interested, that gives the hangupcode. Possibly, we could read that if we find it... Food for thought.
By: Olle Johansson (oej) 2005-07-23 16:08:24

...and please test it :-)
By: Kevin P. Fleming (kpfleming) 2005-07-25 13:33:39

Committed to CVS HEAD with minor mods (compiler warnings about const pointers, and some cleanup in add_header). Thanks!
By: Kevin P. Fleming (kpfleming) 2005-07-25 13:33:55

Oh, and I used MixedCase for the new header name too :-)
By: Digium Subversion (svnbot) 2008-01-15 15:42:22.000-0600

Repository: asterisk
Revision: 6192

U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r6192 | kpfleming | 2008-01-15 15:42:22 -0600 (Tue, 15 Jan 2008) | 2 lines

add plain-text hangup cause private header to outbound BYE/CANCEL (bug ASTERISK-4661 with minor mods)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=6192