From acf329a3c7c814480b8893a189d6dae4ee7401ad Mon Sep 17 00:00:00 2001 From: George Joseph Date: Thu, 25 Feb 2016 14:39:54 -0700 Subject: res_pjsip_mwi: Turn some NOTICEs and WARNINGs into debug 1s. 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 | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'res') diff --git a/res/res_pjsip_mwi.c b/res/res_pjsip_mwi.c index f6600dd63..e1eea6f2a 100644 --- a/res/res_pjsip_mwi.c +++ b/res/res_pjsip_mwi.c @@ -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; } -- cgit v1.2.3