apps/app_voicemail: Demote an ERROR message to a WARNING message
authorMatthew Jordan <mjordan@digium.com>
Sat, 21 Feb 2015 17:36:39 +0000 (17:36 +0000)
committerMatthew Jordan <mjordan@digium.com>
Sat, 21 Feb 2015 17:36:39 +0000 (17:36 +0000)
When using IMAP voicemail with FreePBX, you will often get ERROR messages
complaining about not being able to find a mailbox. This is due to how FreePBX
handles voicemail mailboxes. Unfortunately, app_voicemail has to consider this
a configuration error, as in any other system it would be indicative of
someone misconfiguring their system.

Regardless, a misconfiguration is a WARNING, and not an ERROR. This patch
demotes the message so that system administrators can hopefully reduce some
of the noise in their log files.

Note that in the original patch this was made into a NOTICE, but that's a
too forgiving.

ASTERISK-24790 #close
Reported by: Graham Barnett
patches:
  app_voicemail.c.patch_noise uploaded by Graham Barnett (License 6685)
........

Merged revisions 432098 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 432099 from http://svn.asterisk.org/svn/asterisk/branches/13

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@432100 65c4cc65-6c06-0410-ace0-fbb531ad65f3

apps/app_voicemail.c

index 1102fcd..8e2df02 100644 (file)
@@ -2430,7 +2430,7 @@ static int __messagecount(const char *context, const char *mailbox, const char *
        /* We have to get the user before we can open the stream! */
        vmu = find_user(&vmus, context, mailbox);
        if (!vmu) {
-               ast_log(AST_LOG_ERROR, "Couldn't find mailbox %s in context %s\n", mailbox, context);
+               ast_log(AST_LOG_WARNING, "Couldn't find mailbox %s in context %s\n", mailbox, context);
                return -1;
        } else {
                /* No IMAP account available */