[Home]

Summary:ASTERISK-12470: [patch] Called Party id isn't handled correctly
Reporter:pj (pj)Labels:
Date Opened:2008-07-28 11:16:43Date Closed:2008-08-07 16:11:20
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_skinny
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 2008080700_skinny_calledpartyid.diff
( 1) skinny.png
Description:Dialed number isn't placed into phone call history, instead, here is placed extension, that is reached through asterisk dialplan.
Similarly, when call is connected, on phone display is shown extension pattern from asterisk dialplan, instead of dialed number.
Source of this issue is probably in CallInfoMessage, that asterisk sends to phone and contain invalid info about called party (it contain eg. 's', that is extension reached in asterisk dialplan).


****** ADDITIONAL INFORMATION ******

for ilustration 'CallInfoMessage' containing wrong called party id is attached (it's from packet dump and wireshark).
tried with wifi phone cisco 7920
Comments:By: Michiel van Baak (mvanbaak) 2008-08-07 15:21:57

Patch uploaded.
Tested with 7960 and 7905.

Is this working on the 7920 as well ?

By: pj (pj) 2008-08-07 15:44:34

yes, it's working much better :o)
only small thing: in history is placed only numbers when call is connected, when I leave only ring outgoing calls, or when I call and got progress message (without connecting) from pstn, like "dialed number isn't available", in this case, numbers aren't stored in placed calls history.
I don't know how much work it will need, so it's on you, to fix this or not...
I'm happy yet with your current patch:o)

By: Digium Subversion (svnbot) 2008-08-07 16:11:17

Repository: asterisk
Revision: 136679

U   trunk/channels/chan_skinny.c

------------------------------------------------------------------------
r136679 | mvanbaak | 2008-08-07 16:11:17 -0500 (Thu, 07 Aug 2008) | 8 lines

show correct called party id and also store this to the 'placed calls' list once the call is connected.

(closes issue ASTERISK-12470)
Reported by: pj
Patches:
     2008080700_skinny_calledpartyid.diff uploaded by mvanbaak (license 7)
Tested by: mvanbaak, pj

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

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