]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
res_pjsip_mwi: Turn some NOTICEs and WARNINGs into debug 1s. 15/2315/5
authorGeorge Joseph <george.joseph@fairview5.com>
Thu, 25 Feb 2016 21:39:54 +0000 (14:39 -0700)
committerJoshua Colp <jcolp@digium.com>
Sat, 27 Feb 2016 22:54:10 +0000 (16:54 -0600)
There are a few cases where we're emitting notices or warnings
for things that really need neither, like a client retrying to subscribe
to mwi when they're not conifgured for it.  They get a 404 so there's no
need for non-debug messages.

Change-Id: I05e38a7ff6c2f2521146f4be6a79731b9864e61f

res/res_pjsip_mwi.c

index f6600dd63639569152a268e9dfbee64c83430c90..e1eea6f2ae82cf323329b1ddb27fc61bb7f9e564 100644 (file)
@@ -448,7 +448,7 @@ static void send_unsolicited_mwi_notify(struct mwi_subscription *sub,
 
                contacts = ast_sip_location_retrieve_aor_contacts(aor);
                if (!contacts || (ao2_container_count(contacts) == 0)) {
-                       ast_log(LOG_NOTICE, "No contacts bound to AOR %s. Cannot send unsolicited MWI until a contact registers.\n", aor_name);
+                       ast_debug(1, "No contacts bound to AOR %s. Cannot send unsolicited MWI until a contact registers.\n", aor_name);
                        continue;
                }
 
@@ -600,7 +600,7 @@ static int mwi_validate_for_aor(void *obj, void *arg, int flags)
        mailboxes = ast_strdupa(aor->mailboxes);
        while ((mailbox = strsep(&mailboxes, ","))) {
                if (endpoint_receives_unsolicited_mwi_for_mailbox(endpoint, mailbox)) {
-                       ast_log(LOG_NOTICE, "Endpoint '%s' already configured for unsolicited MWI for mailbox '%s'. "
+                       ast_debug(1, "Endpoint '%s' already configured for unsolicited MWI for mailbox '%s'. "
                                        "Denying MWI subscription to %s\n", ast_sorcery_object_get_id(endpoint), mailbox,
                                        ast_sorcery_object_get_id(aor));
                        return -1;
@@ -710,13 +710,13 @@ static int mwi_new_subscribe(struct ast_sip_endpoint *endpoint,
 
        aor = ast_sip_location_retrieve_aor(resource);
        if (!aor) {
-               ast_log(LOG_WARNING, "Unable to locate aor %s. MWI subscription failed.\n",
+               ast_debug(1, "Unable to locate aor %s. MWI subscription failed.\n",
                        resource);
                return 404;
        }
 
        if (ast_strlen_zero(aor->mailboxes)) {
-               ast_log(LOG_NOTICE, "AOR %s has no configured mailboxes. MWI subscription failed.\n",
+               ast_debug(1, "AOR %s has no configured mailboxes. MWI subscription failed.\n",
                        resource);
                return 404;
        }