Asterisk
  1. Asterisk
  2. ASTERISK-8719

[patch] User has to wait for ResponseTimeout before extension is dialed

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Severity: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: Applications/app_disa
    • Labels:
      None
    • SVN Revision Number:
      54261
    • Mantis ID:
      8971
    • Regression:
      No

      Description

      When a user dials in to app_disa and dials a extension, he has to wait for ResponseTimeout (which could take up to 10 seconds). The user should be able to speed up the dialing process, when he enters '#'. I tested the attached patch and it works. Maybe you want to apply it in the future to app_disa ...

        Activity

        Hide
        Patrick Cervicek added a comment -

        1) disclaimer sent
        2) diff against trunk done

        Show
        Patrick Cervicek added a comment - 1) disclaimer sent 2) diff against trunk done
        Hide
        Leif Madsen added a comment -

        /Housekeeping

        Where are we at with this patch? Is the functionality required?

        Leif.

        Show
        Leif Madsen added a comment - /Housekeeping Where are we at with this patch? Is the functionality required? Leif.
        Hide
        raarts added a comment -

        Corydon76: Maybe you can discriminate which countries are dialled with whatever length of extension, based upon the prefix, but that would require a very long dialplan, which would need to be maintained as national dialplans change.

        I think this is a useful functionality for app_disa.

        Show
        raarts added a comment - Corydon76: Maybe you can discriminate which countries are dialled with whatever length of extension, based upon the prefix, but that would require a very long dialplan, which would need to be maintained as national dialplans change. I think this is a useful functionality for app_disa.
        Hide
        Tilghman Lesher added a comment -

        This really deserves to be optional, so I've gone through and reworked DISA to this end. You'll need to set the 5th argument to DISA() to 'p', if you wish to have this new behavior.

        Patch needs testing.

        Show
        Tilghman Lesher added a comment - This really deserves to be optional, so I've gone through and reworked DISA to this end. You'll need to set the 5th argument to DISA() to 'p', if you wish to have this new behavior. Patch needs testing.
        Hide
        Tilghman Lesher added a comment -

        Committed, revision 68894.

        Show
        Tilghman Lesher added a comment - Committed, revision 68894.

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development