|Summary:||ASTERISK-07428: [patch] Erroneous rename event when blind transfering to parkext|
|Reporter:||Havard Sorbo (hs)||Labels:|
|Date Opened:||2006-07-31 07:42:26||Date Closed:||2006-12-11 08:12:30.000-0600|
|Environment:||Attachments:||( 0) bug-7625.patch|
( 1) log-combined-new.txt
|Description:||When using blind transfer to park a call I get a rename event from the manager api with the Oldname value set to a non-existing channel. (If it exist it hasn' been announced in a newchannel-event)|
****** ADDITIONAL INFORMATION ******
Attacing log containg dialup and parking.
|Comments:||By: Serge Vecher (serge-v) 2006-07-31 10:11:02|
Can you please also attach a respective SIP debug. Please do the following:
1) Prepare test environment (reduce the ammount of unrelated traffic on the server);
2) Make sure your logger.conf has the following line:
console => notice,warning,error,debug
3) restart Asterik.
4) Enable SIP transaction logging with the following CLI commands:
set debug 4
set verbose 4
5) Save complete console log to file and _attach_ said file to the bug.
By: Havard Sorbo (hs) 2006-08-02 09:58:59
Uploaded new debug-log combined with asterisk manager log.
By: Serge Vecher (serge-v) 2006-08-02 10:01:33
as requested, please post the console capture -- debug file clips some important info out.
By: Havard Sorbo (hs) 2006-08-02 11:31:42
Logging from console posted.
Do you want me to file a bug on debug-to-file clipping important info? :P
By: Serge Vecher (serge-v) 2006-08-02 11:52:57
I guess the event in question is:
Since it is preceded by the following event, Oldname: should really by SIPPeer/SIP/803-081a7998<MASQ>
By: Havard Sorbo (hs) 2006-08-02 12:24:42
the renaming is as follows:
SIPPeer/SIP/803-081a7998 -> SIP/803-081a7998
SIP/803-081a7998 -> SIP/803-081a7998<MASQ>
which would be perfectly valid if the channel/name "SIPPeer/SIP/803-081a7998" existed. If you parse the log you see that the first reference to "SIPPeer/SIP..." is in Oldname, which is a bit weird
By: Moises Silva (moy) 2006-09-04 11:12:49
serge-v: I see this situation very similar to the channel created by manager action "Redirect" with prefix name "AsyncGoto/" AND REPORTED with event Newchannel because the chan->_state is set with ast_setstate() before doing the masquerading thing with ast_channel_masquerade() and ast_do_masquerade(). I think the proper thing to do is report this new channel "SIPPeer/SIP/803-0817c610" with a Newchannel event, just as AsyncGoto does. I can make a patch quickly, but need to know if that solves your problem hs?
May be usefull bug references:
By: Moises Silva (moy) 2006-09-04 15:26:51
im uploading a fix for 1.2.11
disclaimer faxed several months ago.
By: Serge Vecher (serge-v) 2006-09-27 13:39:26
hs: did you try moy's patch?
By: jmls (jmls) 2006-11-01 06:46:21.000-0600
ping: hs: did you try moy's patch?
By: Matt O'Gorman (mogorman) 2006-12-08 17:52:48.000-0600
hs if this is still an issue please reopen bug and comment about the patch.