[asterisk-commits] jrose: branch 12 r401938 - in /branches/12/res: ./ res_pjsip/

[SVN commits to the Asterisk project]

Author: jrose
Date: Fri Oct 25 12:34:30 2013
New Revision: 401938

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=401938
Log:
PJSIP: Add log messages when requests are received for non-existent endpoints

(closes issue ASTERISK-22552)
Reported by: Rusty Newton
Review: https://reviewboard.asterisk.org/r/2934/

Modified:
    branches/12/res/res_pjsip/pjsip_distributor.c
    branches/12/res/res_pjsip_endpoint_identifier_user.c

Modified: branches/12/res/res_pjsip/pjsip_distributor.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip/pjsip_distributor.c?view=diff&rev=401938&r1=401937&r2=401938
==============================================================================
--- branches/12/res/res_pjsip/pjsip_distributor.c (original)
+++ branches/12/res/res_pjsip/pjsip_distributor.c Fri Oct 25 12:34:30 2013
@@ -232,6 +232,16 @@
 	return artificial_endpoint;
 }
 
+static void log_unidentified_request(pjsip_rx_data *rdata)
+{
+	char from_buf[PJSIP_MAX_URL_SIZE];
+	char callid_buf[PJSIP_MAX_URL_SIZE];
+	pjsip_uri_print(PJSIP_URI_IN_FROMTO_HDR, rdata->msg_info.from->uri, from_buf, PJSIP_MAX_URL_SIZE);
+	ast_copy_pj_str(callid_buf, &rdata->msg_info.cid->id, PJSIP_MAX_URL_SIZE);
+	ast_log(LOG_NOTICE, "Request from '%s' failed for '%s:%d' (callid: %s) - No matching endpoint found\n",
+		from_buf, rdata->pkt_info.src_name, rdata->pkt_info.src_port, callid_buf);
+}
+
 static pj_bool_t endpoint_lookup(pjsip_rx_data *rdata)
 {
 	struct ast_sip_endpoint *endpoint;
@@ -262,6 +272,7 @@
 			ast_copy_pj_str(name, &sip_from->user, sizeof(name));
 		}
 
+		log_unidentified_request(rdata);
 		ast_sip_report_invalid_endpoint(name, rdata);
 	}
 	rdata->endpt_info.mod_data[endpoint_mod.id] = endpoint;

Modified: branches/12/res/res_pjsip_endpoint_identifier_user.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip_endpoint_identifier_user.c?view=diff&rev=401938&r1=401937&r2=401938
==============================================================================
--- branches/12/res/res_pjsip_endpoint_identifier_user.c (original)
+++ branches/12/res/res_pjsip_endpoint_identifier_user.c Fri Oct 25 12:34:30 2013
@@ -102,6 +102,8 @@
 			return NULL;
 		}
 		ast_debug(3, "Retrieved endpoint %s\n", ast_sorcery_object_get_id(endpoint));
+	} else {
+		ast_debug(3, "Could not identify endpoint by username '%s'\n", endpoint_name);
 	}
 	return endpoint;
 }