[asterisk-bugs] [Asterisk 0014294]: [patch] X-Asterisk-Hangupcause header only in challenged BYEs

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14294 
====================================================================== 
Reported By:                klaus3000
Assigned To:                oej
====================================================================== 
Project:                    Asterisk
Issue ID:                   14294
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 169203 
Request Review:              
====================================================================== 
Date Submitted:             2009-01-21 04:41 CST
Last Modified:              2009-01-29 02:47 CST
====================================================================== 
Summary:                    [patch] X-Asterisk-Hangupcause header only in
challenged BYEs
Description: 
Hi!

The X-Asterisk-Hangupcause is added in transmit_request_with_auth()
function, but not in transmit_request(). Why?

IMO, the Hangupcause header should be added to all BYE requests
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
parent of           0013385 [patch] Reason header support
====================================================================== 

---------------------------------------------------------------------- 
 (0099022) svnbot (reporter) - 2009-01-29 02:47
 http://bugs.digium.com/view.php?id=14294#c99022 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 172169

U   branches/1.4/channels/chan_sip.c

------------------------------------------------------------------------
r172169 | oej | 2009-01-29 02:47:31 -0600 (Thu, 29 Jan 2009) | 16 lines

Make sure that we always add the hangupcause headers. In some cases, the
owner was disconnected before we checked for the cause.
This patch implements a temporary storage in the pvt and use that instead.

The code is based on ideas from code from Adomjan in issue
http://bugs.digium.com/view.php?id=13385 (Add
support for Reason: header)
Thanks to Klaus Darillion for testing!

(closes issue http://bugs.digium.com/view.php?id=14294)
related to issue http://bugs.digium.com/view.php?id=13385

Reported by: klaus3000 and adomjan
Patches: 
      bug14294b.diff uploaded by oej (license 306)
      Based on 20080829_chan_sip.c-q850reason_header.patch uploaded by
adomjan (license 487)
Tested by: oej, klaus3000


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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-29 02:47 svnbot         Checkin                                      
2009-01-29 02:47 svnbot         Note Added: 0099022                          
======================================================================