[Asterisk-code-review] app_meetme: Emit warning if conference not	found. (asterisk[19])
    Friendly Automation 
    asteriskteam at digium.com
       
    Fri Apr  8 11:24:13 CDT 2022
    
    
  
Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/18345 )
Change subject: app_meetme: Emit warning if conference not found.
......................................................................
app_meetme: Emit warning if conference not found.
Currently, if a user tries to access a non-dynamic
MeetMe conference and the conference is not found,
the call simply silent hangs up. There is no indication
to the user that anything went wrong at all.
This changes the relevant debug message to a warning
so that the user is notified of this invalidity.
ASTERISK-29954 #close
Change-Id: Iebcfae3755d00f2150d676ee211c57bc59530048
---
M apps/app_meetme.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Benjamin Keith Ford: Looks good to me, approved
  Friendly Automation: Approved for Submit
diff --git a/apps/app_meetme.c b/apps/app_meetme.c
index 5f498eb..519d1c6 100644
--- a/apps/app_meetme.c
+++ b/apps/app_meetme.c
@@ -4761,7 +4761,7 @@
 				}
 			}
 			if (!var) {
-				ast_debug(1, "%s isn't a valid conference\n", confno);
+				ast_log(LOG_WARNING, "%s isn't a valid conference\n", confno);
 			}
 			ast_config_destroy(cfg);
 		}
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18345
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: 19
Gerrit-Change-Id: Iebcfae3755d00f2150d676ee211c57bc59530048
Gerrit-Change-Number: 18345
Gerrit-PatchSet: 2
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220408/037294d9/attachment.html>
    
    
More information about the asterisk-code-review
mailing list