[Asterisk-code-review] bridge_basic: Change warning to verbose if transfer cancelled (asterisk[master])

N A asteriskteam at digium.com
Tue Aug 24 20:06:35 CDT 2021


N A has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16351 )


Change subject: bridge_basic: Change warning to verbose if transfer cancelled
......................................................................

bridge_basic: Change warning to verbose if transfer cancelled

The attended transfer feature will emit a warning if the user
cancels the transfer or the attended transfer doesn't complete
for any reason. Changes the warning to a verbose message,
since nothing is actually wrong here.

ASTERISK-29612 #close

Change-Id: I64c93cdb21360a0a8d45e9cb6db3af8168f66e6d
---
M main/bridge_basic.c
1 file changed, 4 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/51/16351/1

diff --git a/main/bridge_basic.c b/main/bridge_basic.c
index 21b4b0e..83c9ef5 100644
--- a/main/bridge_basic.c
+++ b/main/bridge_basic.c
@@ -3352,13 +3352,12 @@
 	/* Grab the extension to transfer to */
 	if (grab_transfer(bridge_channel->chan, exten, sizeof(exten), props->context)) {
 		/*
-		 * XXX The warning here really should be removed.  While the
-		 * message is accurate, this is a normal exit for when the user
-		 * fails to specify a valid transfer target.  e.g., The user
+		 * This is a normal exit for when the user fails
+		 * to specify a valid transfer target.  e.g., The user
 		 * hungup, didn't dial any digits, or dialed an invalid
 		 * extension.
 		 */
-		ast_log(LOG_WARNING, "Channel %s: Unable to acquire target extension for attended transfer.\n",
+		ast_verb(3, "Channel %s: Unable to acquire target extension for attended transfer.\n",
 			ast_channel_name(bridge_channel->chan));
 		ast_bridge_channel_write_unhold(bridge_channel);
 		attended_transfer_properties_shutdown(props);
@@ -3661,4 +3660,4 @@
 
 	ast_bridge_features_register(AST_BRIDGE_BUILTIN_ATTENDEDTRANSFER, feature_attended_transfer, NULL);
 	ast_bridge_features_register(AST_BRIDGE_BUILTIN_BLINDTRANSFER, feature_blind_transfer, NULL);
-}
+}
\ No newline at end of file

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16351
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I64c93cdb21360a0a8d45e9cb6db3af8168f66e6d
Gerrit-Change-Number: 16351
Gerrit-PatchSet: 1
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210824/3c725a38/attachment.html>


More information about the asterisk-code-review mailing list