On 01.11.2009 9:49 Uhr, Juha Heinanen wrote:
i started to wonder why some stuff is missing from accounting records
and looks like the reason is that pv $ai is not working.  i have in my
script:

if (is_present_hf("P-Asserted-Identity")) {
    xlog("L_INFO", "P-A-I header <$(hdr(P-Asserted-Identity))> with URI 
<$ai>\n");

and i get to syslog:

Nov 1 10:50:19 localhost /usr/sbin/sip-proxy[1241]: INFO: P-A-I header <sip:[email protected]> with URI <<null>>
it was not merged completely from K 1.5. In Kamailio 3.0 branch I did a fix using a different approach to set header type -- instead of importing the case construct from K 1.5 there is now a function that compare the hdr name and set the type. It is simpler approach, easy to extend and I think the performance impact is minimal. However, we had some discussions on sr-dev and hdr handling in sip_msg structure should be reviewed a bit (e.g., too many hooks for not common used headers, like subject, etc).

If this is ok, I can merge it to sr_3.0 and master. If you prefer the case approach, then you have to import it from K 1.5.

You can import my patch in sr_3.0 via:

git fetch origin
git cherry-pick -x 78f069c3726085902101a55d3bb98f0be7549b60

The patch I did should fix the PPI header as well.

Let me know if works.

Cheers,
Daniel

--
Daniel-Constantin Mierla
* http://www.asipto.com/


_______________________________________________
sr-dev mailing list
[email protected]
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to