Asterisk
  1. Asterisk
  2. ASTERISK-3917

Call Parking times out to the wrong extension

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: Resources/res_features
    • Labels:
      None
    • Mantis ID:
      4036
    • Regression:
      No

      Description

      I'm able to call park just fine, I can pick up a call just fine. but if nobody picks up the call and Asterisk tries to send the call back to te extension that parks it, it fails.

                • ADDITIONAL INFORMATION ******

      001 – Executing NoOp("SIP/0004f201e463-a-7650", "EXTEN=3599 CONTEXT=toll-access") in new stack
      002 – Executing Dial("SIP/0004f201e463-a-7650", "SIP/0004f200cf85-a") in new stack
      003 – Called 0004f200cf85-a
      004 – SIP/0004f200cf85-a-de7a is ringing
      005 – SIP/0004f200cf85-a-de7a answered SIP/0004f201e463-a-7650
      006 – Attempting native bridge of SIP/0004f201e463-a-7650 and SIP/0004f200cf85-a-de7a
      007 – Started music on hold, class 'default', on SIP/0004f201e463-a-7650
      008 – Executing Park("SIP/0004f200cf85-a-bbca", "") in new stack
      009 == Parked SIP/0004f200cf85-a-bbca on 3516. Will timeout back to toll-access,s,1 in 30 seconds
      010 – Playing 'digits/3' (language 'en')
      011 – Playing 'digits/5' (language 'en')
      012 – Playing 'digits/1' (language 'en')
      013 – Playing 'digits/6' (language 'en')
      014 – Added extension '3516' priority 1 to parkedcalls
      015 – Started music on hold, class 'default', on SIP/0004f200cf85-a-bbca
      016 == Spawn extension (toll-access, s, 1) exited KEEPALIVE on 'SIP/0004f200cf85-a-bbca'
      017 – Stopped music on hold on SIP/0004f200cf85-a-bbca
      018 – Stopped music on hold on SIP/0004f201e463-a-7650
      019 – Started music on hold, class 'default', on SIP/0004f201e463-a-7650
      020 == Spawn extension (toll-access, 3599, 2) exited non-zero on 'SIP/0004f200cf85-a-bbca<ZOMBIE>'
      021 == Timeout for SIP/0004f201e463-a-7650 parked on 3516. Returning to toll-access,s,1
      022 – Stopped music on hold on SIP/0004f201e463-a-7650
      023 == Starting SIP/0004f201e463-a-7650 at toll-access,s,1 failed so falling back to exten 's'
      024 == Starting SIP/0004f201e463-a-7650 at toll-access,s,1 still failed so falling back to context 'default'

        Activity

        Hide
        Kevin P. Fleming (Inactive) added a comment -

        People are reporting this does not behave this way in CVS HEAD; can we figure out what the difference is so that stable doesn't get a different fix?

        Show
        Kevin P. Fleming (Inactive) added a comment - People are reporting this does not behave this way in CVS HEAD; can we figure out what the difference is so that stable doesn't get a different fix?
        Hide
        Russell Bryant added a comment -

        My patch basically makes the do_parking_thread have the same behavior of CVS HEAD. I actually started by copying do_parking_thread from CVS HEAD to write the patch.

        In CVS head, we timeout to the peer that parked the call. In 1.0, we timeout to the 's' extension in the context from which the call came. The change came in a commit by jim. For the diff on the web cvs viewer, see http://asterisk.espia-net.net/horde/chora/diff.php/asterisk/res/res_features.c?login=2&r1=1.22&r2=1.23&ty=h

        It may be a good idea to make this optional in CVS head, but for now, I was going to commit a note to UPGRADE.txt to note the change in behavior.

        Show
        Russell Bryant added a comment - My patch basically makes the do_parking_thread have the same behavior of CVS HEAD. I actually started by copying do_parking_thread from CVS HEAD to write the patch. In CVS head, we timeout to the peer that parked the call. In 1.0, we timeout to the 's' extension in the context from which the call came. The change came in a commit by jim. For the diff on the web cvs viewer, see http://asterisk.espia-net.net/horde/chora/diff.php/asterisk/res/res_features.c?login=2&r1=1.22&r2=1.23&ty=h It may be a good idea to make this optional in CVS head, but for now, I was going to commit a note to UPGRADE.txt to note the change in behavior.
        Hide
        Kevin P. Fleming (Inactive) added a comment -

        Now I don't understand how this was ever supposed to work at all...

        Show
        Kevin P. Fleming (Inactive) added a comment - Now I don't understand how this was ever supposed to work at all...
        Hide
        Russell Bryant added a comment -

        ... did this work for you?

        I haven't heard anything since the time when you needed this.

        Show
        Russell Bryant added a comment - ... did this work for you? I haven't heard anything since the time when you needed this.
        Hide
        Russell Bryant added a comment -

        This is not being included in 1.0.

        Personally, I think that the new behavior in CVS HEAD should be optional, since a lot of people probably prefer the "old" way.

        Show
        Russell Bryant added a comment - This is not being included in 1.0. Personally, I think that the new behavior in CVS HEAD should be optional, since a lot of people probably prefer the "old" way.

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development