[asterisk-commits] dlee: branch 12 r399049 - /branches/12/res/

[SVN commits to the Asterisk project]

Author: dlee
Date: Fri Sep 13 09:21:23 2013
New Revision: 399049

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=399049
Log:
res_pjsip: Forward PJSIP logging to Asterisk logging

This patch uses PJSIP's pj_log_set_log_func() to forward PJSIP's log
messages to Asterisk's logger. This is done in a new module:
res_pjsip_log_forwarder.so.

This patch sets defaultenabled on the existing res_pjsip_logger.so to
no, since logging every SIP packet seems a bit odd to do by default, and
is (hopefully) less necessary with regular PJSIP logging.

It also removes res_rtp_asterisk's disabling of PJSIP logging.

(closes issue ASTERISK-22360)
Reported by: Joshua Colp
Review: https://reviewboard.asterisk.org/r/2830/

Added:
    branches/12/res/res_pjsip_log_forwarder.c
      - copied unchanged from r399044, team/dlee/ASTERISK-22360-pjsip-logging/res/res_pjsip_log_forwarder.c
Modified:
    branches/12/res/res_pjsip_logger.c
    branches/12/res/res_rtp_asterisk.c

Modified: branches/12/res/res_pjsip_logger.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip_logger.c?view=diff&rev=399049&r1=399048&r2=399049
==============================================================================
--- branches/12/res/res_pjsip_logger.c (original)
+++ branches/12/res/res_pjsip_logger.c Fri Sep 13 09:21:23 2013
@@ -19,6 +19,7 @@
 /*** MODULEINFO
 	<depend>pjproject</depend>
 	<depend>res_pjsip</depend>
+	<defaultenabled>no</defaultenabled>
 	<support_level>core</support_level>
  ***/
 

Modified: branches/12/res/res_rtp_asterisk.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_rtp_asterisk.c?view=diff&rev=399049&r1=399048&r2=399049
==============================================================================
--- branches/12/res/res_rtp_asterisk.c (original)
+++ branches/12/res/res_rtp_asterisk.c Fri Sep 13 09:21:23 2013
@@ -4537,8 +4537,6 @@
 #ifdef HAVE_PJPROJECT
 	pj_lock_t *lock;
 
-	pj_log_set_level(0);
-
 	if (pj_init() != PJ_SUCCESS) {
 		return AST_MODULE_LOAD_DECLINE;
 	}