[sr-dev] git:master: modules/ims_qos: use asserted_identity for subscription-id and allow e164 as subscription type

Module: sip-router Branch: master Commit: 532459374af8354c030fdce8ff837beab793be6c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5324593… Author: Richard Good <richard.good(a)smilecoms.com> Committer: Richard Good <richard.good(a)smilecoms.com> Date: Tue Feb 11 08:41:27 2014 +0200 modules/ims_qos: use asserted_identity for subscription-id and allow e164 as subscription type use p_asserted_identity as default subscription-id for AAR requests allow E164 to be used as subscription type for AAR requests --- modules/ims_qos/rx_aar.c | 54 ++++++++++++++++++++++++++++++--------------- 1 files changed, 36 insertions(+), 18 deletions(-) diff --git a/modules/ims_qos/rx_aar.c b/modules/ims_qos/rx_aar.c index b5e7293..69d04c0 100644 --- a/modules/ims_qos/rx_aar.c +++ b/modules/ims_qos/rx_aar.c @@ -412,8 +412,9 @@ int rx_send_aar(struct sip_msg *req, struct sip_msg *res, AAAMessage* aar = 0; - - //AAAMessage* aaa = 0; + + str identifier; + int identifier_type; AAA_AVP* avp = 0; @@ -488,15 +489,38 @@ int rx_send_aar(struct sip_msg *req, struct sip_msg *res, } LM_DBG("Adding subscription id...\n"); - /* Add Subscription ID AVP*/ - int identifier_type = AVP_Subscription_Id_Type_SIP_URI; //we only do IMPU now - //to get the SIP URI I use the dlg direction - if its mo I get the from uri from the req, if its mt I get the to uri from the req - str identifier; + //if its mo we use p_asserted_identity in request - if that not there we use from_uri + //if its mt we use p_asserted_identity in reply - if that not there we use to_uri + if (dlg_direction == DLG_MOBILE_ORIGINATING) { - cscf_get_from_uri(req, &identifier); + LM_DBG("originating direction\n"); + if ((identifier = cscf_get_asserted_identity(req)).len == 0) { + LM_DBG("No P-Asserted-Identity hdr found in request. Using From hdr in req"); + + if (!cscf_get_from_uri(req, &identifier)) { + LM_ERR("Error assigning P-Asserted-Identity using From hdr in req"); + goto error; + } + } } else { - cscf_get_to_uri(req, &identifier); - } + LM_DBG("terminating direction\n"); + if ((identifier = cscf_get_asserted_identity(res)).len == 0) { + LM_DBG("No P-Asserted-Identity hdr found in response. Using To hdr in resp"); + + if (!cscf_get_to_uri(res, &identifier)) { + LM_ERR("Error assigning P-Asserted-Identity using To hdr in resp"); + goto error; + } + } + } + + if (strncasecmp(identifier.s,"tel:",4)==0) { + identifier_type = AVP_Subscription_Id_Type_E164; // + }else{ + identifier_type = AVP_Subscription_Id_Type_SIP_URI; //default is END_USER_SIP_URI + } + + rx_add_subscription_id_avp(aar, identifier, identifier_type); LM_DBG("Adding reservation priority...\n"); @@ -552,14 +576,6 @@ int rx_send_aar(struct sip_msg *req, struct sip_msg *res, return ret; - // LM_DBG("sending AAR to PCRF\n"); - // if (rx_forced_peer.len) - // aaa = cdpb.AAASendRecvMessageToPeer(aar, &rx_forced_peer); - // else - // aaa = cdpb.AAASendRecvMessage(aar); - // - // return aaa; - error: LM_ERR("unexpected error\n"); if (aar) @@ -587,6 +603,7 @@ int rx_send_aar_register(struct sip_msg *msg, AAASession* auth, str *ip, int ret = 0; AAA_AVP* avp = 0; char x[4]; + str identifier; LM_DBG("Send AAR register\n"); @@ -614,8 +631,9 @@ int rx_send_aar_register(struct sip_msg *msg, AAASession* auth, str *ip, } /* Add Subscription ID AVP*/ - str identifier; - cscf_get_from_uri(msg, &identifier); + + identifier = cscf_get_public_identity(msg); + int identifier_type = AVP_Subscription_Id_Type_SIP_URI; //we only do IMPU now rx_add_subscription_id_avp(aar, identifier, identifier_type);