Summary:	ASTERISK-11623: Parking timeout fails to return to console and. probably any VoIP device
Reporter:	David Woolley (davidw)	Labels:
Date Opened:	2008-03-12 07:59:29	Date Closed:	2008-03-14 15:05:59
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Resources/res_features
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) Unparking_Problems.txt
Description:	If you call the Park() application, from the console, and let it time out, it tries to return to OSS/dsp not Console/dsp, which fails with an unknown channel type. It also logs a verbose trace line claiming that it is going to return to extension s, whereas this trace line is only valid if you use ParkAndTrace (pass a null peer to the core function). The traces to be attached where taken for another problem (ASTERISK-1212193) and also show issues with parking lot numbers which are not relevant to the current issue. ****** ADDITIONAL INFORMATION ****** Although the console is special in that that technology part of the channel is not the same as that in the app_dial string, I strongly suspect that this applies to anything for which the channel name doesn't match the app_dial string, e.g, because there is a connection instance suffix, or there is a masquerading prefix. I don't know the history and I don't have channel associated hardware to play with, but I would guess that channel names and dial strings may be the same for simple analogue extensions, and this might have worked in the past.  (In several places channel is used where app_dial string is meant.) As it is now, I suspect it almost never works. Visual inspection of the latest trunk version (106757) suggests that the only change in this area is that /'s in the temporary extension entry used are replaced by 0, but the dial string is still the raw peer channel name.
Comments:	By: David Woolley (davidw) 2008-03-12 13:57:17 ASTERISK-1083890 also reports the problem with the verbose message on all but ParkAndAnnounce. By: David Woolley (davidw) 2008-03-12 16:28:59 Looking at the trace from ASTERISK-1083890, it looks like the session id gets stripped from SIP channel names. Further examination indicates that transfers to parkingext are special-cased in the SIP channel, so may not be valid indicators of the behaviour for other ways of getting a SIP channel parked. By: Digium Subversion (svnbot) 2008-03-14 15:05:24 Repository: asterisk Revision: 108796 U   branches/1.4/channels/chan_oss.c ------------------------------------------------------------------------ r108796 | russell | 2008-03-14 15:05:22 -0500 (Fri, 14 Mar 2008) | 5 lines Fix a channel name issue.  chan_oss registers the "Console" channel type, but it created channels with an "OSS" prefix. (closes issue ASTERISK-11623, reported by davidw, patched by me) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=108796 By: Digium Subversion (svnbot) 2008-03-14 15:05:49 Repository: asterisk Revision: 108797 _U  trunk/ U   trunk/channels/chan_oss.c ------------------------------------------------------------------------ r108797 | russell | 2008-03-14 15:05:47 -0500 (Fri, 14 Mar 2008) | 13 lines Merged revisions 108796 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r108796 | russell | 2008-03-14 15:09:22 -0500 (Fri, 14 Mar 2008) | 5 lines Fix a channel name issue.  chan_oss registers the "Console" channel type, but it created channels with an "OSS" prefix. (closes issue ASTERISK-11623, reported by davidw, patched by me) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=108797 By: Digium Subversion (svnbot) 2008-03-14 15:05:59 Repository: asterisk Revision: 108798 _U  branches/1.6.0/ U   branches/1.6.0/channels/chan_oss.c ------------------------------------------------------------------------ r108798 | russell | 2008-03-14 15:05:56 -0500 (Fri, 14 Mar 2008) | 21 lines Merged revisions 108797 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r108797 | russell | 2008-03-14 15:09:37 -0500 (Fri, 14 Mar 2008) | 13 lines Merged revisions 108796 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r108796 | russell | 2008-03-14 15:09:22 -0500 (Fri, 14 Mar 2008) | 5 lines Fix a channel name issue.  chan_oss registers the "Console" channel type, but it created channels with an "OSS" prefix. (closes issue ASTERISK-11623, reported by davidw, patched by me) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=108798