Asterisk
  1. Asterisk
  2. ASTERISK-4195

[patch] [post 1.2] Bridge of 2 existing channels including Manager API

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Severity: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: Resources/res_features
    • Labels:
      None
    • Mantis ID:
      4297
    • Regression:
      No

      Description

      The patches below(for releases 1.0.6, 1.0.7 and CVS-HEAD) allow the Bridging of two existing channels wither through the Manager API or through a function call. I use MASQ to dump the two existing channels into something I can manipulate then use ast_bridge_call to bridge the calls in a separate thread, using the same method that a call is consultatively transferred.

      This code does sometimes cause a couple WARNINGs upon execution, but has never caused crashes or corruption of channels. Maybe a more experienced Asterisk coder can figure out how to streamline the patch somewhat.

      Example Manager API Action send:

      Action: Bridge
      ChannelA: Zap/1-1
      ChannelB: SIP/cc100-cd91

        Activity

        Hide
        Serge Vecher added a comment -

        Looks like this bug needs some attention: we have positive reviews here ...

        Show
        Serge Vecher added a comment - Looks like this bug needs some attention: we have positive reviews here ...
        Hide
        Denis Voitenko added a comment -

        The latest patch fails on CVS-HEAD 11/15/05

        patch -p0 < action-bridge.txt
        patching file res_features.c
        patch: **** malformed patch at line 27: ast_channel_alloc(0);

        Show
        Denis Voitenko added a comment - The latest patch fails on CVS-HEAD 11/15/05 patch -p0 < action-bridge.txt patching file res_features.c patch: **** malformed patch at line 27: ast_channel_alloc(0);
        Hide
        gnudialer gnudialer (Inactive) added a comment -

        patch works for me on CVS HEAD 11/22/05 (when i checked it out)

        Show
        gnudialer gnudialer (Inactive) added a comment - patch works for me on CVS HEAD 11/22/05 (when i checked it out)
        Hide
        xiribitata added a comment -

        I've tested this feature with the latest patch (action_bridge-updated-10-12) and it seems that it isn't sending any kind of response to the manager api (neither positive or negative). It is only sending the Bridge event, but this is an event, it doesn't have the action id of the request previously made, so i can't map it with the request.

        Shouldn't it be sending a positive or negative response to the request as well as the event?

        Show
        xiribitata added a comment - I've tested this feature with the latest patch (action_bridge-updated-10-12) and it seems that it isn't sending any kind of response to the manager api (neither positive or negative). It is only sending the Bridge event, but this is an event, it doesn't have the action id of the request previously made, so i can't map it with the request. Shouldn't it be sending a positive or negative response to the request as well as the event?
        Hide
        Matt O'Gorman (Inactive) added a comment -

        all current work is being done on ASTERISK-5689

        Show
        Matt O'Gorman (Inactive) added a comment - all current work is being done on ASTERISK-5689

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development