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/General
    • Labels:
      None
    • Mantis ID:
      3836
    • Regression:
      No

      Description

      ChanSpy is back! (and this time it's personal) </cheesy announcer voice>

      Here it is at long last!

      show application ChanSpy

      = Info about application 'ChanSpy' =

      [Synopsis]:
      Tap into any type of asterisk channel and listen to audio

      [Description]:
      Chanspy([<scanspec>][|<options>])

      Valid Options:

      • q: quiet, don't announce channels beep, etc.
      • b: bridged, only spy on channels involved in a bridged call.
      • v([-4..4]): adjust the initial volume. (negative is quieter)
      • g: enforce group. read $ {SPYGROUP}

        and only match calls also in this group.

      If <scanspec> is specified, only channel names beginning with that string will be scanned.
      ('all' or an empty string are also both valid <scanspec>)

      While Spying:

      Dialing # cycles the volume level.
      Dialing * will stop spying and look for another channel to spy on.
      Dialing a series of digits followed by # builds a channel name to append to <scanspec>
      (e.g. run Chanspy(Agent) and dial 1234# while spying to jump to channel Agent/1234)

                • ADDITIONAL INFORMATION ******

      Disclamer on file
      anthmct@yahoo.com <--valid paypal address wink

        Activity

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

        Not that it matters here, but just for future reference...

        There is no value in using a 'char' for status in struct ast_channel_spy, since it is surrounded by word-size variables (that require word alignment) it will end up taking 4 bytes anyway.

        Ideally, you'd do this:

        enum chanspy_states

        { CHANSPY_NEW = 0, CHANSPY_RUNNING = 1, CHANSPY_DONE = 2 }

        ;

        struct ast_channel_spy

        { ... enum chanspy_states status; ... }

        ;

        This will still require an int, but now it's obvious that is a limited set of valid values.

        Show
        Kevin P. Fleming (Inactive) added a comment - Not that it matters here, but just for future reference... There is no value in using a 'char' for status in struct ast_channel_spy, since it is surrounded by word-size variables (that require word alignment) it will end up taking 4 bytes anyway. Ideally, you'd do this: enum chanspy_states { CHANSPY_NEW = 0, CHANSPY_RUNNING = 1, CHANSPY_DONE = 2 } ; struct ast_channel_spy { ... enum chanspy_states status; ... } ; This will still require an int, but now it's obvious that is a limited set of valid values.
        Hide
        Anthony Minessale added a comment -

        I like enum but nothing else uses it so I didnt want to break protocol
        maybe a patch to make everything into enums should emerge.

        Show
        Anthony Minessale added a comment - I like enum but nothing else uses it so I didnt want to break protocol maybe a patch to make everything into enums should emerge.
        Hide
        Kevin P. Fleming (Inactive) added a comment -

        Well, I used them to good effect in the QUEUESTATUS patch, so that's a start

        Show
        Kevin P. Fleming (Inactive) added a comment - Well, I used them to good effect in the QUEUESTATUS patch, so that's a start
        Hide
        Mark Spencer added a comment -

        Added to CVS, thanks!

        Show
        Mark Spencer added a comment - Added to CVS, thanks!
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 5242

        U trunk/apps/Makefile
        A trunk/apps/app_chanspy.c

        ------------------------------------------------------------------------
        r5242 | markster | 2008-01-15 15:28:26 -0600 (Tue, 15 Jan 2008) | 2 lines

        Add chanspy (bug ASTERISK-3746)

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=5242

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 5242 U trunk/apps/Makefile A trunk/apps/app_chanspy.c ------------------------------------------------------------------------ r5242 | markster | 2008-01-15 15:28:26 -0600 (Tue, 15 Jan 2008) | 2 lines Add chanspy (bug ASTERISK-3746 ) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5242

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development