[sr-dev] git:jason.penton/ims_ro_interface: modules/ims_ro: more logging improvements

[Jason Penton]

Module: sip-router
Branch: jason.penton/ims_ro_interface
Commit: 3b33ef509f58f229e163996354fbdd2b7c4d7c9f
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3b33ef509f58f229e163996354fbdd2b7c4d7c9f

Author: Jason Penton <jason.penton at smilecoms.com>
Committer: Jason Penton <jason.penton at smilecoms.com>
Date:   Wed Mar 20 10:41:04 2013 +0200

modules/ims_ro: more logging improvements

---

 modules/ims_ro/ccr.c    |    4 ++--
 modules/ims_ro/ims_ro.c |    6 ++----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/modules/ims_ro/ccr.c b/modules/ims_ro/ccr.c
index 4fba04a..ef5a093 100644
--- a/modules/ims_ro/ccr.c
+++ b/modules/ims_ro/ccr.c
@@ -274,7 +274,7 @@ Ro_CCA_t *Ro_parse_CCA_avps(AAAMessage *cca) {
                 mscc_avp_list_ptr = &mscc_avp_list;
                 AAA_AVP *mscc_avp = mscc_avp_list_ptr->head;
                 while (mscc_avp != NULL) {
-                    LM_DBG("MSCC AVP code is [%i] and data lenght is [%i]", mscc_avp->code, mscc_avp->data.len);
+                    LM_DBG("MSCC AVP code is [%i] and data length is [%i]", mscc_avp->code, mscc_avp->data.len);
                     switch (mscc_avp->code) {
                             AAA_AVP_LIST y;
                             AAA_AVP *z;
@@ -286,7 +286,7 @@ Ro_CCA_t *Ro_parse_CCA_avps(AAAMessage *cca) {
                                     case AVP_CC_Time:
                                         mscc->granted_service_unit->cc_time = get_4bytes(z->data.s);
                                     default:
-                                        LM_ERR("Unsupported Granted Service Unit.\n");
+                                        LM_ERR("Unsupported Granted Service Unit with code:[%d]\n", z->code);
                                 }
                                 z = z->next;
                             }
diff --git a/modules/ims_ro/ims_ro.c b/modules/ims_ro/ims_ro.c
index 9c34721..173b172 100644
--- a/modules/ims_ro/ims_ro.c
+++ b/modules/ims_ro/ims_ro.c
@@ -643,8 +643,8 @@ void send_ccr_stop(struct ro_session *ro_session) {
 
     event_type = new_event_type(&sip_method, &sip_event, 0);
 
-    LM_DBG("Sending CCR STOP request for for user [%.*s] using session id [%.*s]",
-    		ro_session->from_uri.len, ro_session->from_uri.s, ro_session->ro_session_id.len, ro_session->ro_session_id.s);
+    LM_DBG("Sending CCR STOP request for for user:[%.*s] using session id:[%.*s] and units:[%d]\n",
+    		ro_session->from_uri.len, ro_session->from_uri.s, ro_session->ro_session_id.len, ro_session->ro_session_id.s, used);
 
     req_timestamp = time(0);
 
@@ -722,8 +722,6 @@ void send_ccr_stop(struct ro_session *ro_session) {
         LM_ERR("problem add Termination cause AVP to STOP record.\n");
     }
 
-    LM_DBG("Sending CCR Diameter message for STOP record on Ro Sesison ID [%.*s] with charge units [%d]\n",
-    		ro_session->ro_session_id.len, ro_session->ro_session_id.s, used);
     /* send sunchronously so we can respond to callplan (cfg file), so a decision can be made to process the invite */
     cdp_avp->cdp->AAASessionsUnlock(auth->hash);
     AAAMessage *cca = cdp_avp->cdp->AAASendRecvMessageToPeer(acr, &cfg.destination_host);