[Asterisk-code-review] app_bridgeaddchan.c: Make BridgeAdd be more like Bridge (asterisk[master])
George Joseph
asteriskteam at digium.com
Tue Jan 7 14:29:31 CST 2020
George Joseph has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/13517 )
Change subject: app_bridgeaddchan.c: Make BridgeAdd be more like Bridge
......................................................................
app_bridgeaddchan.c: Make BridgeAdd be more like Bridge
* Made BridgeAdd not hangup the call if there is a problem.
* Reduced message level from warning to verbose for normal exception
cases.
* Added a loop safety check to BridgeAdd.
* Made BridgeAdd set BRIDGERESULT with the status when dialplan is
resumed.
Change-Id: I374d39b8a3edcc794eeb5c6b9f31a01424cdc426
---
M apps/app_bridgeaddchan.c
A doc/CHANGES-staging/app_bridgeaddchan_add_bridgeresult_var.txt
A doc/UPGRADE-staging/app_bridgeaddchan_add_bridgeresult_var.txt
3 files changed, 56 insertions(+), 20 deletions(-)
Approvals:
Sean Bright: Looks good to me, but someone else must approve
Joshua Colp: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved; Approved for Submit
diff --git a/apps/app_bridgeaddchan.c b/apps/app_bridgeaddchan.c
index 34642a6..6080868 100644
--- a/apps/app_bridgeaddchan.c
+++ b/apps/app_bridgeaddchan.c
@@ -43,17 +43,28 @@
Join a bridge that contains the specified channel.
</synopsis>
<syntax>
- <parameter name="name">
- <para>Name of the channel in an existing bridge
- </para>
+ <parameter name="channel" required="true">
+ <para>The current channel joins the bridge containing the channel
+ identified by the channel name, channel name prefix, or channel
+ uniqueid.</para>
</parameter>
</syntax>
<description>
<para>This application places the incoming channel into
the bridge containing the specified channel. The specified
channel only needs to be the prefix of a full channel name
- IE. 'SIP/cisco0001'.
+ IE. 'PJSIP/cisco0001'.
</para>
+ <para>This application sets the following channel variable upon completion:</para>
+ <variablelist>
+ <variable name="BRIDGERESULT">
+ <para>The result of the bridge attempt as a text string.</para>
+ <value name="SUCCESS" />
+ <value name="FAILURE" />
+ <value name="LOOP" />
+ <value name="NONEXISTENT" />
+ </variable>
+ </variablelist>
</description>
</application>
***/
@@ -66,15 +77,28 @@
struct ast_bridge_features chan_features;
struct ast_bridge *bridge;
char *c_name;
+ int failed;
/* Answer the channel if needed */
if (ast_channel_state(chan) != AST_STATE_UP) {
ast_answer(chan);
}
- if (!(c_ref = ast_channel_get_by_name_prefix(data, strlen(data)))) {
- ast_log(LOG_WARNING, "Channel %s not found\n", data);
- return -1;
+ if (ast_strlen_zero(data)) {
+ data = "";
+ c_ref = NULL;
+ } else {
+ c_ref = ast_channel_get_by_name_prefix(data, strlen(data));
+ }
+ if (!c_ref) {
+ ast_verb(4, "Channel '%s' not found\n", data);
+ pbx_builtin_setvar_helper(chan, "BRIDGERESULT", "NONEXISTENT");
+ return 0;
+ }
+ if (chan == c_ref) {
+ ast_channel_unref(c_ref);
+ pbx_builtin_setvar_helper(chan, "BRIDGERESULT", "LOOP");
+ return 0;
}
c_name = ast_strdupa(ast_channel_name(c_ref));
@@ -86,26 +110,24 @@
ast_channel_unref(c_ref);
if (!bridge) {
- ast_log(LOG_WARNING, "Channel %s is not in a bridge\n", c_name);
- return -1;
+ ast_verb(4, "Channel '%s' is not in a bridge\n", c_name);
+ pbx_builtin_setvar_helper(chan, "BRIDGERESULT", "FAILURE");
+ return 0;
}
- ast_verb(3, "%s is joining %s in bridge %s\n", ast_channel_name(chan),
- c_name, bridge->uniqueid);
+ ast_verb(4, "%s is joining %s in bridge %s\n",
+ ast_channel_name(chan), c_name, bridge->uniqueid);
- if (ast_bridge_features_init(&chan_features)
- || ast_bridge_join(bridge, chan, NULL, &chan_features, NULL, 0)) {
-
- ast_log(LOG_WARNING, "%s failed to join %s in bridge %s\n", ast_channel_name(chan),
- c_name, bridge->uniqueid);
-
- ast_bridge_features_cleanup(&chan_features);
- ao2_cleanup(bridge);
- return -1;
+ failed = ast_bridge_features_init(&chan_features)
+ || ast_bridge_join(bridge, chan, NULL, &chan_features, NULL, 0);
+ if (failed) {
+ ast_verb(4, "%s failed to join %s in bridge %s\n",
+ ast_channel_name(chan), c_name, bridge->uniqueid);
}
ast_bridge_features_cleanup(&chan_features);
ao2_cleanup(bridge);
+ pbx_builtin_setvar_helper(chan, "BRIDGERESULT", failed ? "FAILURE" : "SUCCESS");
return 0;
}
diff --git a/doc/CHANGES-staging/app_bridgeaddchan_add_bridgeresult_var.txt b/doc/CHANGES-staging/app_bridgeaddchan_add_bridgeresult_var.txt
new file mode 100644
index 0000000..784e502
--- /dev/null
+++ b/doc/CHANGES-staging/app_bridgeaddchan_add_bridgeresult_var.txt
@@ -0,0 +1,7 @@
+Subject: app_bridgeaddchan
+Master-Only: true
+
+The BridgeAdd application now behaves more like the Bridge application.
+The application now sets the BRIDGERESULT channel variable to indicate
+what happened when the channel resumes in dialplan. This is instead of
+hanging up the channel on failure conditions.
diff --git a/doc/UPGRADE-staging/app_bridgeaddchan_add_bridgeresult_var.txt b/doc/UPGRADE-staging/app_bridgeaddchan_add_bridgeresult_var.txt
new file mode 100644
index 0000000..784e502
--- /dev/null
+++ b/doc/UPGRADE-staging/app_bridgeaddchan_add_bridgeresult_var.txt
@@ -0,0 +1,7 @@
+Subject: app_bridgeaddchan
+Master-Only: true
+
+The BridgeAdd application now behaves more like the Bridge application.
+The application now sets the BRIDGERESULT channel variable to indicate
+what happened when the channel resumes in dialplan. This is instead of
+hanging up the channel on failure conditions.
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/13517
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I374d39b8a3edcc794eeb5c6b9f31a01424cdc426
Gerrit-Change-Number: 13517
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200107/8b70f4db/attachment-0001.html>
More information about the asterisk-code-review
mailing list