Asterisk
  1. Asterisk
  2. ASTERISK-26862

app_queue: Queue stops calling members with local interface after forwarding in previous call

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Major Major
    • Resolution: Fixed
    • Affects Version/s: 13.14.0
    • Target Release Version/s: 13.15.0, 14.4.0, 15.0.0
    • Component/s: Applications/app_queue
    • Security Level: None
    • Labels:
      None
    • Frequency of Occurrence:
      Constant

      Description

      I found a problem very similar to ASTERISK-26299, which was not fixed there.

      It will occur if queue calls a member, using local channel, and device dialed is different from device observed for hints.

      Let me modify the example from ASTERISK-26299.

      I would have extensions.conf with:

      [default]
      exten => 100,1,NoOp()
      same => n,Queue(foo)
      same => n,Hangup()
      
      [agents]
      exten => alice,1,NoOp()
      same => n,Wait(5)
      same => n,Dial(SIP/carol)    ; Alice is forwarding all her calls to Carol
      same => n,Hangup()
      

      And the following queues.conf:

      [foo]
      strategy = rrmemory
      timeout = 30
      ringinuse = 0
      
      member => Local/alice@agents,0,Alice,SIP/alice
      

      Bob calls 100 and gets connected to Carol. When Bob calls 100 again no one is called and debug log shows:

      DEBUG[6846][C-00000004]: app_queue.c:4447 ring_one: Trying 'Local/alice@agents' with metric 1000
      DEBUG[6846][C-00000004]: app_queue.c:4232 can_ring_entry: Local/alice@agents has another call trying, can't receive call
      

      In this situation, in order to remove SIP/alice from pending members, queue will wait until SIP/alice state is changed to "not in use", which will never happen. Only SIP/carol will experience state changes.

        Issue Links

          Activity

          Hide
          Asterisk Team added a comment -

          Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

          A good first step is for you to review the Asterisk Issue Guidelines if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

          Then, if you are submitting a patch, please review the Patch Contribution Process.

          Show
          Asterisk Team added a comment - Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution. A good first step is for you to review the Asterisk Issue Guidelines if you haven't already. The guidelines detail what is expected from an Asterisk issue report. Then, if you are submitting a patch, please review the Patch Contribution Process .
          Hide
          Robert Mordec added a comment -

          Attached file queue_local_fwd_13.14.patch fixed the problem for me.

          Show
          Robert Mordec added a comment - Attached file queue_local_fwd_13.14.patch fixed the problem for me.
          Hide
          Joshua Colp added a comment -

          If you'd like to go through the contribution process linked on the initial comment we can review and see if this is the correct way to resolve this. If not then it's up to someone else to go through the process which can take some time.

          Show
          Joshua Colp added a comment - If you'd like to go through the contribution process linked on the initial comment we can review and see if this is the correct way to resolve this. If not then it's up to someone else to go through the process which can take some time.
          Hide
          Robert Mordec added a comment -

          I removed the attached file with patch that was rejected during code review.

          Show
          Robert Mordec added a comment - I removed the attached file with patch that was rejected during code review.
          Hide
          Friendly Automation added a comment -

          Change 5210 merged by Joshua Colp:
          app_queue: Member stuck as pending after forwarding previous call from queue

          https://gerrit.asterisk.org/5210

          Show
          Friendly Automation added a comment - Change 5210 merged by Joshua Colp: app_queue: Member stuck as pending after forwarding previous call from queue https://gerrit.asterisk.org/5210
          Hide
          Friendly Automation added a comment -

          Change 5211 merged by zuul:
          app_queue: Member stuck as pending after forwarding previous call from queue

          https://gerrit.asterisk.org/5211

          Show
          Friendly Automation added a comment - Change 5211 merged by zuul: app_queue: Member stuck as pending after forwarding previous call from queue https://gerrit.asterisk.org/5211
          Hide
          Friendly Automation added a comment -

          Change 5212 merged by zuul:
          app_queue: Member stuck as pending after forwarding previous call from queue

          https://gerrit.asterisk.org/5212

          Show
          Friendly Automation added a comment - Change 5212 merged by zuul: app_queue: Member stuck as pending after forwarding previous call from queue https://gerrit.asterisk.org/5212
          Hide
          Friendly Automation added a comment -

          Change 5248 merged by Joshua Colp:
          app_queue: Member stuck as pending after forwarding previous call from queue

          https://gerrit.asterisk.org/5248

          Show
          Friendly Automation added a comment - Change 5248 merged by Joshua Colp: app_queue: Member stuck as pending after forwarding previous call from queue https://gerrit.asterisk.org/5248

            People

            • Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development