Summary:ASTERISK-12109: [patch] parking space number is announced to the parkee instead of the parker
Reporter:Norbert Reinartz (nreinartz)Labels:
Date Opened:2008-05-29 10:37:11Date Closed:2008-08-29 12:59:32
Versions:Frequency of
Environment:Attachments:( 0) asterisk-parking_timeout-0.0.1.patch
( 1) configuration.txt
Description:If call parking is done by res/res_features.c, park_call_exec() the parking space number is announced to the parkee but it should be announced to the parker.
Revision 97529 ( http://svn.digium.com/view/asterisk/branches/1.4/res/res_features.c?view=log#rev97529    ) introduced a fix and stated out "Fix saying the parking space number to the caller doing the parking ..." but its not working as described.
Until revision 97529 the parking space wasn't announced. Now its announced, but it's announced to the parkee not to the parker.

Most of the channel drivers do parking directly by ast_park_call() if the number matches with ast_parking_ext(). If the number doesn't match, ast_async_goto() is called which goes through the dial plan.
ast_async_goto() calls the registered application park_call_exec(). Unfortunately this function doesn't include the peer channel as parameter. Thus it's not possible to announce the parking space number to the parker. A design with both channels as parameters would be great.

I tested with some snom phones using the park orbit function. Usually parking is done by ast_park_call(), I had to use a special configuration, so that parking is done by park_call_exec(). See attached file "configuration.txt".

The attached patch doesn't solve the whole problem, but with this patch the parker is at least called back after parking timeout.
I tested this for chan_sip, it works for me. The patch also undoes the fix of revision 97529.
Comments:By: Jeff Peeler (jpeeler) 2008-08-29 12:58:37

I've looked at trying to make this work and have determined it is just not going to be possible. Park was not really designed to be used directly in the dialplan with a blind transfer. If you really want to do a blind transfer to park the call, use the registered parking extension (parkext) from features.conf which is handled appropriately.

As a partial compromise, this bug inspired the addition of a new argument to the Park application. The new option 's' will silence playing the parking number. It'll be in 1.6.1 once it's released.